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

Updating loader-utils to v3.2.0 #120

Closed
wants to merge 1 commit into from

Conversation

sam-glendenning
Copy link
Contributor

Description

We recently had a couple of PRs on OperationsGateway and DataGateway created by Dependabot to resolve critical vulnerabilities recently discovered in v2 and v1 of loader-utils respectively. I have since merged both PRs but neither project is using the latest version of loader-utils.

This PR aims to address that by upgrading to v3. It may be I'm unaware of why we're using an older, deprecated version (perhaps it's still a transitive dependency for many of our dependencies) so if that's the case, we can close this PR. Otherwise, we can upgrade and consider doing the same with DataGateway.

Testing instructions

Check latest version is being used.

  • Review code
  • Check Actions build
  • Review changes to test coverage

Agile board tracking

Hotfix

@codecov
Copy link

codecov bot commented Nov 8, 2022

Codecov Report

Base: 93.80% // Head: 93.80% // No change to project coverage 👍

Coverage data is based on head (eebc755) compared to base (46f7d26).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #120   +/-   ##
========================================
  Coverage    93.80%   93.80%           
========================================
  Files           49       49           
  Lines         1793     1793           
  Branches       446      446           
========================================
  Hits          1682     1682           
  Misses         111      111           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@sam-glendenning
Copy link
Contributor Author

My conclusion is that this is fundamentally incompatible with some essential libraries we use, therefore I'm happy to stick with v2 which, while deprecated, is still heavily used in the community. I'll close this PR.

@sam-glendenning sam-glendenning deleted the update-loader-utils-to-v3 branch November 8, 2022 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant