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

Change testing/* to use single quotes #1419

Open
tonyandrewmeyer opened this issue Oct 10, 2024 · 1 comment
Open

Change testing/* to use single quotes #1419

tonyandrewmeyer opened this issue Oct 10, 2024 · 1 comment
Labels
low priority Non-urgent issue we may (or may not) consider later small item testing Related to ops.testing

Comments

@tonyandrewmeyer
Copy link
Contributor

To be consistent with the rest of the code, we should change all of testing/**/*.py to use ' in preference to ", as we did with the other code earlier in the year.

The testing/pyproject.toml configuration should be altered to remove the special case, then ruff can be run, and then we'll want to add the commit to the blame ignore list.

@dimaqq
Copy link
Contributor

dimaqq commented Oct 11, 2024

bike shed time: let's take lead from scneario and switch to double quotes 🙄

@benhoyt benhoyt added the testing Related to ops.testing label Oct 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
low priority Non-urgent issue we may (or may not) consider later small item testing Related to ops.testing
Projects
None yet
Development

No branches or pull requests

3 participants