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

Revert "Revert "add rvagg as admin to go-merkledag and go-cid"" #94

Closed
wants to merge 1 commit into from

Conversation

galargh
Copy link
Contributor

@galargh galargh commented Dec 5, 2022

Reverts #93

@galargh galargh requested review from a team as code owners December 5, 2022 17:27
@galargh galargh requested a review from laurentsenta December 5, 2022 17:27
@willscott
Copy link
Contributor

i assume this is testing the automation / required CI job setup. It looks like maybe when the required check is skipped rather than failed github doesn't prevent merging?
image

@galargh
Copy link
Contributor Author

galargh commented Dec 6, 2022

i assume this is testing the automation / required CI job setup. It looks like maybe when the required check is skipped rather than failed github doesn't prevent merging? image

You're right! I created ipdxco/github-as-code#89 to address this. Thank you Will for helping uncover this. I'll keep an eye out if we don't do a similar mistake in other, non-github-mgmt repos.

Screenshot 2022-12-06 at 12 16 53

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