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

deps: simplify dependencies #95792

Merged
merged 1 commit into from
Jan 25, 2023
Merged

Conversation

knz
Copy link
Contributor

@knz knz commented Jan 24, 2023

This upgrades cockroachdb/errors, which in turn drops a bunch of old, now-unused indirect dependencies.

This change is valuable because some of these non-used dependencies were marked in security vulnerability databases and creating false positive reports.

Release note: None
Epic: None

@cockroach-teamcity
Copy link
Member

This change is Reviewable

@knz knz requested a review from rickystewart January 24, 2023 21:58
Copy link
Contributor

@sjbarag sjbarag left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gotta love a red diff 🙂

Thanks for handling this!

This upgrades cockroachdb/errors, which in turn drops a bunch
of old, now-unused indirect dependencies.

This change is valuable because some of these non-used dependencies
were marked in security vulnerability databases and creating
false positive reports.

Release note: None
@knz knz requested a review from a team as a code owner January 24, 2023 22:46
@knz
Copy link
Contributor Author

knz commented Jan 24, 2023

TFYR!

bors r=sjbarag

@craig
Copy link
Contributor

craig bot commented Jan 25, 2023

Build succeeded:

@craig craig bot merged commit 782f94a into cockroachdb:master Jan 25, 2023
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.

3 participants