-
Notifications
You must be signed in to change notification settings - Fork 27
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
feature: fix linting problems #156
Labels
Comments
5 tasks
leon1995
added a commit
to leon1995/sdc11073
that referenced
this issue
Aug 31, 2023
5 tasks
leon1995
added a commit
that referenced
this issue
Sep 6, 2023
part of linting fix #156 . this fixes `RUF012` ## Types of changes <!--- What types of changes does your code introduce? Put an `x` in all the boxes that apply: --> - [ ] Bug fix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to change) ## Checklist: <!--- Go over all the following points, and put an `x` in all the boxes that apply. --> <!--- If you're unsure about any of these, don't hesitate to ask. We're here to help! --> - [ ] I have updated the [changelog](../CHANGELOG.md) accordingly. - [ ] I have added tests to cover my changes.
leon1995
added a commit
that referenced
this issue
Oct 5, 2023
<!--- Provide a general summary of your changes in the title above --> <!--- Link the corresponding issues after you created the pull request --> fixes [TID252](https://docs.astral.sh/ruff/rules/relative-imports/) can be verified with `ruff check . --select TID252` #156 ## Types of changes <!--- What types of changes does your code introduce? Put an `x` in all the boxes that apply: --> - [ ] Bug fix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to change) ## Checklist: <!--- Go over all the following points, and put an `x` in all the boxes that apply. --> <!--- If you're unsure about any of these, don't hesitate to ask. We're here to help! --> - [ ] I have updated the [changelog](../CHANGELOG.md) accordingly. - [ ] I have added tests to cover my changes.
5 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Is there an existing issue for this?
Feature idea
fix issues found by linter
Alternatives
Implementation idea
No response
Participation
Link to the idea discussion
No response
The text was updated successfully, but these errors were encountered: