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.

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

Jump to bottom

Use ruff check $FILES instead of ruff $FILES #23

Merged
merged 1 commit into from
Mar 8, 2024
Merged

Use ruff check $FILES instead of ruff $FILES #23

merged 1 commit into from
Mar 8, 2024

Conversation

facutuesca
Copy link
Contributor

Running make lint or make reformat shows the following warning:

warning: `ruff <path>` is deprecated. Use `ruff check <path>` instead.

The invocation without the check subcommand has been deprecated in version 0.3.0.

This changes the Makefile to fix that.

@facutuesca
Use ruff check $FILES instead of ruff $FILES
bd6479d 
The invocation without the `check` subcommand has been deprecated
in version 0.3.0.
@facutuesca facutuesca requested a review from woodruffw March 8, 2024 16:34
woodruffw
woodruffw approved these changes Mar 8, 2024
View reviewed changes
Copy link
Member

@woodruffw woodruffw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@woodruffw woodruffw merged commit 6e492ea into main Mar 8, 2024
4 checks passed
@woodruffw woodruffw deleted the fix-deprecated-ruff-invocation branch March 8, 2024 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@woodruffw woodruffw woodruffw approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@facutuesca @woodruffw