-
Notifications
You must be signed in to change notification settings - Fork 856
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
Fix dependency vulnerabilites #1723
Conversation
Thanks for the requested dependency upgrades, @MLobo1997! We'll wait to approve until you mark this PR as no longer in draft, but then we'd be happy to merge in changes like these. |
Hey @bhancock8, what's happened with the CI of this repo? There seem to be no longer any builds on circleci |
Hmm, interesting. 🤔 We'll take a look. Thanks for flagging @MLobo1997. |
That would be great, thanks @bhancock8! If you need anything let me know. I'll polish this PR once the builds are back in place. |
@MLobo1997 Thanks for flagging this—from what I can tell, there seems to be something amiss on the CircleCI side of things. I’ve contacted them and am working toward a resolution. Will update you when I’ve heard back! |
@MLobo1997 Builds should be working again! The CircleCI auth keys had expired just recently, so CircleCI wasn't able to pickup the config. Let me know if it isn't working for you still. |
This pull request is stale because it has been open 90 days with no activity. Remove stale label or comment or this will be closed in 7 days. |
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.
Pretty straightforward bump of dependencies. Looks good to me!
Description of proposed changes
The security score on Snyk is very low
Related issue(s)
Fixes # (issue)
Test plan
Checklist
Need help on these? Just ask!
tox -e complex
and/ortox -e spark
if appropriate.