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

Format with ruff #390

Merged
merged 2 commits into from
Jun 4, 2024
Merged

Format with ruff #390

merged 2 commits into from
Jun 4, 2024

Conversation

LionelColaso
Copy link
Collaborator

eliminate ruff related complaints

eliminate ruff related complaints
@LionelColaso LionelColaso requested a review from hk21702 June 4, 2024 06:34
@github-actions github-actions bot added the DevOps 💻 Related to development automation, auto building, auto actions, and other CI/CD things label Jun 4, 2024
Copy link
Collaborator

@hk21702 hk21702 left a comment

Choose a reason for hiding this comment

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

Checks out for me.

Added lxml-stubs to dev requirements since I noticed it was an error

@hk21702 hk21702 added the Code style ✨ Minor changes like fixing typos, style, indentation label Jun 4, 2024
@LionelColaso LionelColaso merged commit 36dea62 into main Jun 4, 2024
8 of 9 checks passed
@LionelColaso LionelColaso deleted the format_with_ruff branch June 4, 2024 06:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code style ✨ Minor changes like fixing typos, style, indentation DevOps 💻 Related to development automation, auto building, auto actions, and other CI/CD things
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants