Skip to content

add ruff lint step to workflow #135

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

add ruff lint step to workflow #135

Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
d814d3a
add ruff lint step to workflow
ChrisBarker-NOAA Dec 20, 2024
e67e42e
uv add ruff before using it.
ChrisBarker-NOAA Dec 20, 2024
bd1f600
fix syntax for running ruff
ChrisBarker-NOAA Dec 20, 2024
5b403a7
put the needs check back the way it was.
ChrisBarker-NOAA Dec 20, 2024
d19d178
removed lint step to see if it works again.
ChrisBarker-NOAA Dec 20, 2024
cce07ae
ng lint stanza back in.
ChrisBarker-NOAA Dec 20, 2024
1f7d84b
added lint to check
ChrisBarker-NOAA Dec 20, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 15 additions & 1 deletion .github/workflows/main.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,10 +23,24 @@ jobs:
- name: Run tests
run: scripts/test

lint:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4

- name: Install uv
uses: astral-sh/setup-uv@v4
# run: uv add ruff

- name: ruff check
run: uv run --with ruff ruff check bson/

# https://github.com/marketplace/actions/alls-green#why used for branch protection checks
check:
if: always()
needs: [test]
needs: [test, lint]
# - test
# - lint
runs-on: ubuntu-latest
steps:
- name: Decide whether the needed jobs succeeded or failed
Expand Down
Loading