fix(app): Bump dependencies (testcontainers & prepare for Rust Edition 2024, remove MongoDB) #121
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
schoenenberg
force-pushed
the
fix/bump-dependencies
branch
from
a93a462 to
a144d17
Compare
|
During the bumping of the dependencies, it was discovered that the MongoDB implementation is not working anymore. Therefore it was decided, to remove this implementation. |
schoenenberg
changed the title
"cargo fmt --check" Result:"cargo clippy" Result:No linter warnings |
|
|
|
Clearinghouse App Build reportFormatter report ("cargo fmt --check"):No formatter warnings Linter report ("cargo clippy"):No linter warnings Test logTest log: |
|
Clearinghouse App Build reportFormatter report ("cargo fmt --check"):Exit code: No formatter warnings Linter report ("cargo clippy"):Exit code: No linter warnings Test logExit code: 0 Test log: |
Clearinghouse App Build reportFormatter report ("cargo fmt --check"):Exit code: 1 Linter report ("cargo clippy"):Exit code: 0 No linter warnings Test logExit code: 0 Test log: |
|
The new |
schoenenberg
force-pushed
the
fix/bump-dependencies
branch
from
44a85bc to
fb32797
Compare
|
Clearinghouse App Build reportFormatter report ("cargo fmt --check"):Exit code: 1 Linter report ("cargo clippy"):Exit code: 0 No linter warnings Test logExit code: 0 Test log: |
Clearinghouse App Build reportFormatter report ("cargo fmt --check"):Exit code: 0 No formatter warnings Linter report ("cargo clippy"):Exit code: 0 No linter warnings Test logExit code: 0 Test log: |
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What this PR changes/adds
Briefly describe WHAT your pr changes, which features it adds/modifies.
Updates
testcontainersdependency and updates version constraints for other dependencies.Why it does that
Briefly state why the change was necessary.
testcontainershad some API changes, to keep up-to-date the version was upgraded and the code appropriately changed.Cargo.tomlwere updated.std::env::set_varandstd::env::remove_varbecomesunsafe(see this tracking issue: Tracking Issue for Rust 2024: Makestd::env::{set_var, remove_var}unsafe rust-lang/rust#124866). To prepare for this, the calls to these functions were put intounsafeblocks and the lint level has been changed fromforbidtodeny, excluding these specific calls withallowdirectives.Further notes
List other areas of code that have changed but are not necessarily linked to the main feature. This could be method signature changes, package declarations, bugs that were encountered and were fixed inline, etc.
Linked Issue(s)
Closes # <-- insert Issue number if one exists