Skip to content

Commit

Permalink
Review feedback
Browse files Browse the repository at this point in the history
  • Loading branch information
KyleFromNVIDIA committed Jan 24, 2025
1 parent 367bc95 commit fe85d85
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions src/rapids_pre_commit_hooks/codeowners.py
Original file line number Diff line number Diff line change
Expand Up @@ -201,9 +201,7 @@ def check_codeowners_line(
codeowners_line.file.pos,
f"file '{codeowners_line.file.filename}' has incorrect owners",
)
extra_string = ""
for missing_required_owner in missing_required_owners:
extra_string += f" {missing_required_owner}"
extra_string = " " + " ".join(missing_required_owners)
last = codeowners_line.owners[-1].pos[1]
warning.add_replacement((last, last), extra_string)

Expand Down

0 comments on commit fe85d85

Please sign in to comment.