Feature to both lint and check format in a single action

[augustelalande]

I would like an option to also check formatting in addition to the default lint check. As far as I can tell this is not possible even playing with the args as it is today (I'm aware of astral-sh/ruff#8232).

My ideal invocation would be

- uses: astral-sh/ruff-action@v2
  with:
    check_format: true

I'm willing to contribute it if people find it useful.

[eifinger]

You can currently achieve this with either using the installed ruff

- uses: astral-sh/ruff-action@v2
- run: ruff format

or using the action several times.

- uses: astral-sh/ruff-action@v2
- uses: astral-sh/ruff-action@v2
  with:
    args: "format --check"

I am hesitant to include a temporary workaround in this action with astral-sh/ruff#8232 already being discussed, because it means more maintenance overhead, possible breaking changes etc.

[daveisfera]

Might not be worth the trouble, but another way to consider it is that the action could currently abstract away the fact that it's 2 commands and then there wouldn't be a need for a workflow change once ruff support doing both natively