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

Use ruff check in Makefile #21138

Merged
merged 1 commit into from
May 1, 2024
Merged

Use ruff check in Makefile #21138

merged 1 commit into from
May 1, 2024

Conversation

charliermarsh
Copy link
Contributor

Summary

ruff /path/to/file.py works but is deprecated, and we now recommend ruff check /path/to/file.py (to match ruff format /path/to/file.py).

Copy link

vercel bot commented May 1, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
langchain ⬜️ Ignored (Inspect) Visit Preview May 1, 2024 5:53am

@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label May 1, 2024
@charliermarsh charliermarsh changed the title Use ruff check in Makefile Use ruff check in Makefile May 1, 2024
@dosubot dosubot bot added the 🤖:nit Small modifications/deletions, fixes, deps or improvements to existing code or docs label May 1, 2024
@dosubot dosubot bot added the lgtm PR looks good. Use to confirm that a PR is ready for merging. label May 1, 2024
@eyurtsev eyurtsev merged commit 2b10c4d into langchain-ai:master May 1, 2024
11 checks passed
@eyurtsev
Copy link
Collaborator

eyurtsev commented May 1, 2024

Thank you @charliermarsh

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm PR looks good. Use to confirm that a PR is ready for merging. 🤖:nit Small modifications/deletions, fixes, deps or improvements to existing code or docs size:XS This PR changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants