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

dep: bump a bunch of dependencies and set some as workspace-wide #607

Merged
merged 6 commits into from
Dec 21, 2024

Conversation

V0ldek
Copy link
Member

@V0ldek V0ldek commented Dec 16, 2024

Short description

Using the magic of workspace-scoped dependencies we now keep some shared deps like log and the internal crates synced across the workspace. This is bundled with a mass-update of deps and fix of a clippy lint that was preventing dependabot from updating the crate for a few weeks now.

  • Bump clap from 4.5.21 to 4.5.23
  • Remove nom as a direct dependency of rsonpath-lib

Checklist

All of these should be ticked off before you submit the PR.

  • I ran just verify locally and it succeeded.
  • Issue was given go ahead and is linked above OR I have included justification for a minor change.
  • Unit tests for my changes are included OR no functionality was changed.

V0ldek and others added 6 commits December 16, 2024 17:36
Using the magic of workspace-scoped dependencies we now keep some
shared deps like `log` and the internal crates synced across the
workspace. This is bundled with a mass-update of deps and fix of
a clippy lint that was preventing dependabot from updating the crate
for a few weeks now.

- Bump clap from 4.5.21 to 4.5.23
- Remove `nom` as a direct dependency of `rsonpath-lib`
seems that dependency-review hits more endpoints for egress now,
switching the policy to audit to see what's changed and update the
policy later
whitelisted the two additional APIs used by dependency-review
@V0ldek V0ldek merged commit 65c7160 into main Dec 21, 2024
50 checks passed
@V0ldek V0ldek deleted the v0ldek/dep-updates branch December 21, 2024 16:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant