search

ChartBoost / ruff-action

A GitHub Action for Ruff
Apache License 2.0
121 stars 21 forks source link

Is it possible to only fix/format the `diff` in the PR? #21

Closed ma-sadeghi closed 3 months ago

brucearctor commented 5 months ago

Probably? Seems like that could be an option that could be added. Is that something you'd be willing to contribute?

ma-sadeghi commented 5 months ago

Sure, I can take a crack at this this weekend

ma-sadeghi commented 5 months ago

Just an update: I do want to contribute, but it might not happen this weekend due to other commitments, just to adjust expectations :)

brucearctor commented 5 months ago

All good. We're happy to accept contributions on whatever timelines are doable for people doing the [ volunteer ] work. hope you're getting into something fun over the weekend -

Vaibhav-Lodha commented 5 months ago

@brucearctor wanted to understand 2 things. I might be able to contribute if time persists in the upcoming weeks -

  1. Currently, when we open a pull request, the default action runs on all files, and to build support for it, we need to change the current implementation, i.e. adding some flag like only git-diff
  2. Secondly, we don't have to functionality like how Django Doctor or gitleaks comments in the file for the errors, is that right, all I could find is a output file in git action?