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: bump logtags #72906

Merged
merged 1 commit into from
Nov 18, 2021
Merged

deps: bump logtags #72906

merged 1 commit into from
Nov 18, 2021

Conversation

knz
Copy link
Contributor

@knz knz commented Nov 18, 2021

needed for #72836.

Needed to be able to do AddTags(..., FromContext(ctx)) when
the ctx might not have tags yet.

Release note: None

@knz knz requested review from stevendanna and tbg November 18, 2021 10:52
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@knz
Copy link
Contributor Author

knz commented Nov 18, 2021

TFYR!

bors r=tbg

@knz
Copy link
Contributor Author

knz commented Nov 18, 2021

bors retry

@craig
Copy link
Contributor

craig bot commented Nov 18, 2021

Already running a review

@knz
Copy link
Contributor Author

knz commented Nov 18, 2021

bors r-

@craig
Copy link
Contributor

craig bot commented Nov 18, 2021

Canceled.

@knz
Copy link
Contributor Author

knz commented Nov 18, 2021

merging under #72836

Needed to be able to do `AddTags(..., FromContext(ctx))` when
the ctx might not have tags yet.

Release note: None
@craig craig bot merged commit 7f58980 into cockroachdb:master Nov 18, 2021
@knz knz deleted the 20211117-ctx branch November 18, 2021 14:37
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