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

Fix the ruff command error #520

Merged
merged 1 commit into from
Sep 2, 2024
Merged

Conversation

Raj725
Copy link
Collaborator

@Raj725 Raj725 commented Aug 30, 2024

Description:
The old command fails with newer versions of ruff. The new command is backward compatible and works with older(ruff 0.4.2) and newer(ruff 0.6.3) Ruff versions.

1. Before and After output for new Ruff version(ruff 0.6.3, New Python 3.9 venv):

Screenshot 2024-08-30 at 10 09 05 AM

2. Before and After output for older Ruff version (ruff 0.4.2, Existing Python 3.9 venv):

Screenshot 2024-08-30 at 10 32 54 AM

@Raj725 Raj725 requested a review from srics August 30, 2024 05:11
@Raj725 Raj725 force-pushed the makefile/updt-format-cmd branch from 9b2b93f to 0b0f78f Compare September 2, 2024 04:52
@srics srics force-pushed the makefile/updt-format-cmd branch from 0b0f78f to aa3a88f Compare September 2, 2024 21:02
Description:
The old command fails with newer versions of ruff.
The new command is backward compatible and works with both older and newer versions of ruff.
@srics srics force-pushed the makefile/updt-format-cmd branch from aa3a88f to 76c43cd Compare September 2, 2024 21:03
@srics srics merged commit 6ad535e into daxa-ai:main Sep 2, 2024
3 checks passed
@Raj725 Raj725 deleted the makefile/updt-format-cmd branch September 10, 2024 05:08
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.

2 participants