Skip to content
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.

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

chore(ci): fix ruff format checking #915

Merged
merged 3 commits into from Apr 17, 2024
Merged

Conversation

jnussbaum
Copy link
Collaborator

until now, the ruff format . in the CI checks didn't really check anything. Ruff must be called with the --check flag in order to exit with non-0 if a wrong formatting is found.

@jnussbaum jnussbaum self-assigned this Apr 17, 2024
@jnussbaum jnussbaum enabled auto-merge (squash) April 17, 2024 11:50
@jnussbaum jnussbaum changed the title chore: fix ruff formatting fix(ci): fix ruff format checking Apr 17, 2024
@jnussbaum jnussbaum disabled auto-merge April 17, 2024 11:51
@jnussbaum jnussbaum changed the title fix(ci): fix ruff format checking chore(ci): fix ruff format checking Apr 17, 2024
@jnussbaum jnussbaum merged commit 270dc6f into main Apr 17, 2024
10 of 12 checks passed
@jnussbaum jnussbaum deleted the wip/fix-ruff-formatting branch April 17, 2024 12:34
@daschbot daschbot mentioned this pull request Apr 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants