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.

Sign up for GitHub

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

Jump to bottom

Add @mrwojtek to CODEOWNERS for cirq/docs/qcvv #3877

Merged
merged 1 commit into from
Mar 4, 2021
Merged

Add @mrwojtek to CODEOWNERS for cirq/docs/qcvv #3877

merged 1 commit into from
Mar 4, 2021

Conversation

mpharrigan
Copy link
Collaborator

No description provided.

@mpharrigan mpharrigan requested a review from balopat March 4, 2021 23:24
@mpharrigan mpharrigan requested review from cduck, vtomole and a team as code owners March 4, 2021 23:24
@google-cla google-cla bot added the cla: yes Makes googlebot stop complaining. label Mar 4, 2021
tanujkhattar
tanujkhattar approved these changes Mar 4, 2021
View reviewed changes
Copy link
Collaborator

@tanujkhattar tanujkhattar left a comment

Choose a reason for hiding this comment

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

LGTM

@balopat balopat added the automerge Tells CirqBot to sync and merge this PR. (If it's running.) label Mar 4, 2021
balopat
balopat approved these changes Mar 4, 2021
View reviewed changes
Copy link
Contributor

@balopat balopat left a comment

Choose a reason for hiding this comment

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

You'll have to fix the codeowners test

@CirqBot CirqBot added the front_of_queue_automerge CirqBot uses this label to indicate (and remember) what's being merged next. label Mar 4, 2021
@CirqBot CirqBot merged commit af6500e into master Mar 4, 2021
@CirqBot CirqBot removed the automerge Tells CirqBot to sync and merge this PR. (If it's running.) label Mar 4, 2021
@CirqBot CirqBot deleted the mpharrigan-patch-1 branch March 4, 2021 23:43
@CirqBot CirqBot removed the front_of_queue_automerge CirqBot uses this label to indicate (and remember) what's being merged next. label Mar 4, 2021
@mpharrigan
Copy link
Collaborator Author

should we be concerned that the codeowners test wasn't failing?

mpharrigan added a commit that referenced this pull request Mar 4, 2021
@mpharrigan
Update codeowners test following #3877
82f3c18 
Update codeowners test following #3877
@mpharrigan mpharrigan mentioned this pull request Mar 4, 2021
Merged
@mpharrigan
Copy link
Collaborator Author

see #3878

CirqBot pushed a commit that referenced this pull request Mar 9, 2021
@mpharrigan
Update codeowners test following #3877 (#3878)
0c2a65c 
Update codeowners test following #3877
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@balopat balopat balopat approved these changes

@tanujkhattar tanujkhattar tanujkhattar approved these changes

@cduck cduck Awaiting requested review from cduck

@vtomole vtomole Awaiting requested review from vtomole

Assignees
No one assigned
Labels
cla: yes Makes googlebot stop complaining.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@mpharrigan @balopat @tanujkhattar @CirqBot