-
Notifications
You must be signed in to change notification settings - Fork 40
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
chore: Update Badger version to v4 #1740
chore: Update Badger version to v4 #1740
Conversation
Codecov ReportPatch and project coverage have no change.
@@ Coverage Diff @@
## develop #1740 +/- ##
===========================================
- Coverage 75.42% 75.42% -0.00%
===========================================
Files 208 208
Lines 21801 21801
===========================================
- Hits 16443 16442 -1
Misses 4210 4210
- Partials 1148 1149 +1
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 2 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
question: Has the fork been reviewed? |
Initial review has been done and it's not going to be the way to fix this but the badger team is also looking into it. Hopefully we get a proper fix before the end of this cycle. |
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.
Requesting changes as I don't think we should get into the habit of committing personal-fork dependencies into the codebase
07b60c7
to
8c50398
Compare
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, thanks for switching to the source fork :)
## Relevant issue(s) Resolves sourcenetwork#1702 ## Description This PR updates the Badger version to v4. It includes a temporary fix to our badger `os.Exit` issue that is costing us time when managing PRs. This should be reverted or properly fixed before releasing v0.7.
Relevant issue(s)
Resolves #1702
Description
This PR updates the Badger version to v4.
It includes a temporary fix to our badger
os.Exit
issue that is costing us time when managing PRs. This should be reverted or properly fixed before releasing v0.7.Tasks
How has this been tested?
multiple runs of
env DEFRA_DETECT_DATABASE_CHANGES=true gotestsum -- ./tests/integration/query/one_to_many_to_one -shuffle=on -p 1
Specify the platform(s) on which this was tested: