-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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: rename/upgrade xdg/scram -> xdg-go/scram #74510
Conversation
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.
Anything interesting here @HonoreDB?
Thanks matt. bors r=shermanCRL |
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; but merge conflicts already cropped up
Release note: None
f968437
to
156f986
Compare
Canceled. |
thanks! bors r=rafiss,shermanCRL |
Pretty weird to me there are no changes to |
The new dependency was already mirrored somehow. I wasn't able to understand why, but it's there. |
@rickystewart that reminds me, old/now-unused dependencies do not get removed from DEPS.bzl, is that expected? |
Build succeeded: |
67221: deps: update bufbuild/buf r=rickystewart a=knz First commit from #74510. Release note: None Co-authored-by: Raphael 'kena' Poss <[email protected]>
Prereq for #74301
Release note: None