Skip to content

Commit 36688e0

Browse files
committed
Changing Ruff action to be checks only
1 parent 5c72c56 commit 36688e0

File tree

2 files changed

+6
-4
lines changed

2 files changed

+6
-4
lines changed

.github/workflows/lint_and_format.yml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,13 +7,15 @@ jobs:
77
steps:
88
- uses: actions/checkout@v4
99
- uses: chartboost/ruff-action@v1
10+
name: Lint
1011
with:
1112
version: 0.3.5
12-
args: check --fix --output-format=full --statistics
13+
args: "check --output-format=full --statistics"
1314
- uses: chartboost/ruff-action@v1
15+
name: Format
1416
with:
1517
version: 0.3.5
16-
args: format
18+
args: "format --check"
1719
- uses: stefanzweifel/git-auto-commit-action@v5
1820
with:
1921
commit_message: "lint fixes and formatting by ruff"

README.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -258,7 +258,7 @@ NOTE: Be sure to merge the latest from "upstream" before making a pull request!
258258

259259
### Checklist Before Pull Request (Optional)
260260

261-
1. Use `ruff check` to check for lint errors
261+
1. Use `ruff check --fix` to check and fix lint errors
262262
2. Use `ruff format` to apply formatting
263263

264-
NOTE: Ruff linting and formatting are done automatically when PR is raised using Git Action (and changes will be automatically applied via another commit). It is, however, a good practice to check and fix lint errors, as well as apply formatting before PR.
264+
NOTE: Ruff linting and formatting checks are done when PR is raised via Git Action. Before raising a PR, it is a good practice to check and fix lint errors, as well as apply formatting.

0 commit comments

Comments
 (0)