Skip to content

Avoid f-string in queries #33

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
wants to merge 1 commit into from
Closed

Avoid f-string in queries #33

wants to merge 1 commit into from

Conversation

@gadomski
Copy link
Member

@gadomski gadomski commented May 11, 2023

Related Issue(s):

Description:

PR Checklist:

  • pre-commit hooks pass locally
  • Tests pass (run make test)
  • Documentation has been updated to reflect changes, if applicable, and docs build successfully (run make docs)
  • Changes are added to the CHANGELOG.

@gadomski gadomski requested a review from pjhartzell May 11, 2023 12:11
@gadomski gadomski self-assigned this May 11, 2023
@gadomski gadomski added this to the 2.4.6 milestone May 11, 2023
@gadomski gadomski closed this May 11, 2023
@gadomski
Copy link
Member Author

gadomski commented May 11, 2023

This had already been picked up by an earlier backport.

@gadomski gadomski deleted the backport/554-avoid-fstring branch May 11, 2023 12:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pjhartzell pjhartzell Awaiting requested review from pjhartzell

Assignees

@gadomski gadomski

Labels

None yet

Projects

None yet

Milestone

2.4.6

Development

Successfully merging this pull request may close these issues.

3 participants

@gadomski @alukach