-
Notifications
You must be signed in to change notification settings - Fork 516
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
⬆️ Upgrade test and lint dependencies #2939
⬆️ Upgrade test and lint dependencies #2939
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Just need a lock file regen but I think these updates are good |
b182b9e
to
d29dc11
Compare
d29dc11
to
fed9316
Compare
Not sure why Snyk Container is failing. @WadeBarnes I notice #2951 was recently merged to fix it, but for some reason that PR didn't have the Snyk Container job run as part of its checks |
Running into the annoying PRs don't have access to secrets and things needing account tokens to work issues again. Was afraid that was going to happen. I'll configure the Synk Container job to continue on error until we get it sorted. |
Decided to switch the job back to running on push for now. That way it's running until we can sort things for PRs. |
@WadeBarnes aah that can be tricky... Thanks! Will rebase once that's in |
The update to the Snyk Container job has been merged. |
fed9316
to
1dfe856
Compare
Signed-off-by: ff137 <[email protected]>
Signed-off-by: ff137 <[email protected]>
Signed-off-by: ff137 <[email protected]>
Signed-off-by: ff137 <[email protected]>
Signed-off-by: ff137 <[email protected]>
Signed-off-by: ff137 <[email protected]>
Signed-off-by: ff137 <[email protected]>
Signed-off-by: ff137 <[email protected]>
Signed-off-by: ff137 <[email protected]>
Signed-off-by: ff137 <[email protected]>
Signed-off-by: ff137 <[email protected]>
Signed-off-by: ff137 <[email protected]>
Signed-off-by: ff137 <[email protected]>
Signed-off-by: ff137 <[email protected]>
Signed-off-by: ff137 <[email protected]>
1dfe856
to
2b6c229
Compare
Quality Gate passedIssues Measures |
⬆️ Upgrades:
pytest
: 8.0.0 -> 8.2.0pytest-asyncio
: 0.23.5 -> 0.23.6pytest-cov
: 4.1.0 -> 5.0.0pytest-ruff
: 0.1.1 -> 0.3.2➖ Removes:
mock
Edit: additional dependency upgrades from #2940 are pulled in here to speed up merge cycle
Therefore includes:
⬆️ Upgrades:
black
: 24.3.0 -> 24.4.2ruff
: 0.1.2 -> 0.4.4pre-commit
: 3.3.3 -> 3.7.1