chore: Update flake8 - #175
spanglerco wants to merge 2 commits into
Conversation
rbell517
left a comment
There was a problem hiding this comment.
This is already covered by the flake8 update in #168 which updates to flake8 v7 but keeps the major version pinned. That is my preference so we don't have wildly different flake8 behavior based on which version the developer might happen to have in their venv. That PR is basically ready to go in I just haven't found the time to address the couple comments and resolve the merge conflicts. Would you be willing to push that one over the finish line instead of continuing with this PR? I know your other PR is blocked on this, so I can be flexible.
Yeah, I can take a look at the other PR. I can incorporate the |
|
Superseded by #168. |
What does this Pull Request accomplish?
Updates flake8 and related dependencies to latest. Stop pinning to specific versions per NI convention.
Why should this Pull Request be merged?
Resolves a bug affecting #174, where flake8 is flagging colons and commas within format strings.
What testing has been done?
Ran flake8 locally. Relying on PR workflow.