Skip to content

Commit

Permalink
ruff: use extend-include option for non-*.py files
Browse files Browse the repository at this point in the history
Per suggestion in astral-sh/ruff#13122
  • Loading branch information
praiskup committed Aug 28, 2024
1 parent dee173f commit 62f4e65
Showing 1 changed file with 11 additions and 2 deletions.
13 changes: 11 additions & 2 deletions vcs-diff-lint
Original file line number Diff line number Diff line change
Expand Up @@ -185,10 +185,19 @@ class RuffLinter(_Linter):
def is_compatible(self, file):
return file.type == 'python'

def command(self, projectdir, _filenames):
def command(self, projectdir, filenames):
cmd = [CSDIFF_RUFF]
# Construct --config 'extend-include=["a", "b", ...]' option
# per https://github.com/astral-sh/ruff/issues/13122
explicitly_include = [fn for fn in filenames if not fn.endswith("*.py")]
if explicitly_include:
extend_include = ','.join([f'"{fn}"' for fn in explicitly_include])
extend_include = f'extend-include=[{extend_include}]'
cmd += ["--config", extend_include]

# While we optimize other analyzers and we scan only the changed
# set of files, Ruff is fast enough to do full-projectdir scan.
cmd = [CSDIFF_RUFF] + [os.path.join(self.gitroot, projectdir)]
cmd.append(os.path.join(self.gitroot, projectdir))
return cmd, {}


Expand Down

0 comments on commit 62f4e65

Please sign in to comment.