chore: Pin dependencies for linters #397
Conversation
lavaleri
force-pushed
the
pin-linter-deps-2.x
branch
from
6475dff to
838a3c2
Compare
| @@ -0,0 +1,14 @@ | |||
| bandit==1.7.0 | |||
There was a problem hiding this comment.
This is a nitpick, but can we consistently use _ or - for file and directory names?
There was a problem hiding this comment.
In this repo, there is a mostly consistent policy that directories use _ and files use -.
I do not see the value in this policy.
I do not know how this came to be.
I am not going to suggest we enforce it in any way.
But since this already meets the policy, I suggest we leave it as is.
| @@ -0,0 +1,14 @@ | |||
| bandit==1.7.0 | |||
There was a problem hiding this comment.
In this repo, there is a mostly consistent policy that directories use _ and files use -.
I do not see the value in this policy.
I do not know how this came to be.
I am not going to suggest we enforce it in any way.
But since this already meets the policy, I suggest we leave it as is.
3 participants
Pin dependencies for linters to minimize broken builds on old branch
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Check any applicable: