Skip to content

Adjust F541 linting issue on Lib/test module #103805

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
Closed
rodolfomp123 opened this issue Apr 24, 2023 · 0 comments
Closed

Adjust F541 linting issue on Lib/test module #103805

rodolfomp123 opened this issue Apr 24, 2023 · 0 comments
Labels
type-feature A feature request or enhancement

Comments

@rodolfomp123
Copy link
Contributor

rodolfomp123 commented Apr 24, 2023 •
edited by bedevere-bot
Loading

Feature or enhancement

After running ruff in CPython, some linting and formatting errors were found.
This issue intends to fix the F541 f-string without any placeholders errors related to the Lib/test module.

Pitch

In the CPython/Lib directory, we run the following command:

ruff test/ --select F541

We can see these kind of alerts:

...
test/test_sqlite3/test_regression.py:494:32: F541 f-string without any placeholders
test/test_sqlite3/test_regression.py:501:32: F541 f-string without any placeholders
test/test_sqlite3/test_regression.py:508:32: F541 f-string without any placeholders
test/test_sqlite3/test_userfunctions.py:565:22: F541 f-string without any placeholders
...

Previous discussion

None

Linked PRs
@rodolfomp123 rodolfomp123 added the type-feature A feature request or enhancement label Apr 24, 2023
@rodolfomp123 rodolfomp123 mentioned this issue Apr 24, 2023
Merged
gpshead added a commit that referenced this issue Apr 24, 2023
@rodolfomp123 @blurb-it @gpshead
GH-103805: Lib test f541 linting issue fix (#103812)
8291ae3 
This PR makes some minor linting adjustments to the Lib/test module
caught by [ruff](https://github.com/charliermarsh/ruff). The adjustments
are all related to the `F541 f-string without any placeholders` issue.

Issue: #103805

<!-- gh-issue-number: gh-103805 -->
* Issue: gh-103805
<!-- /gh-issue-number -->

---------

Co-authored-by: blurb-it[bot] <43283697+blurb-it[bot]@users.noreply.github.com>
Co-authored-by: Gregory P. Smith <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type-feature A feature request or enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@rodolfomp123