Skip to content

Adds the codeowners action for offering the ability to merge PRs#899

Merged
orta merged 1 commit intov2from
codeowners
Aug 13, 2020
Merged

Adds the codeowners action for offering the ability to merge PRs#899
orta merged 1 commit intov2from
codeowners

Conversation

@orta
Copy link
Copy Markdown
Contributor

@orta orta commented Aug 13, 2020

Using https://github.com/orta/code-owner-self-merge to let people merge PRs to their translations without waiting on me

/cc @khaosdoctor @danilofuchs @sasurau4 @Quramy @Naturalclar @Takepepe @KingDarBoja @Kingwl

@github-actions
Copy link
Copy Markdown
Contributor

Messages
📖

Deployed to a PR branch - playground tsconfig old handbook

Generated by 🚫 dangerJS against 66f23d5

@orta orta merged commit b58be92 into v2 Aug 13, 2020
@danilofuchs
Copy link
Copy Markdown
Contributor

@orta I tried leaving a review at #892 with LGTM, but no Action was triggered

@orta
Copy link
Copy Markdown
Contributor Author

orta commented Aug 13, 2020

That's a review, not a comment - I'll add support for that

@orta
Copy link
Copy Markdown
Contributor Author

orta commented Aug 13, 2020

It should handle that now, assuming I understand the semver in GH actions

@danilofuchs
Copy link
Copy Markdown
Contributor

I could get it to trigger by commenting, but it looks like it is checking to see if the PR creator has merge access, instead of the approver:
https://github.com/microsoft/TypeScript-Website/runs/982690050?check_suite_focus=true

@khaosdoctor
Copy link
Copy Markdown
Contributor

Awesome! This will make things a LOT easier!

@khaosdoctor
Copy link
Copy Markdown
Contributor

khaosdoctor commented Aug 14, 2020

I reviewed #815 but it seems not to merge 🤔

--- EDIT

I commented again, it merged. Does it only works on single comments?

danilofuchs added a commit to danilofuchs/TypeScript-Website that referenced this pull request Aug 14, 2020
The new codeowners merge action is awesome! (See microsoft#899)
Playground examples are written in TS, so codeowners should be able to merge them.

(This action failed because of these missing permissions: https://github.com/microsoft/TypeScript-Website/pull/870/checks?check_run_id=985887055, microsoft#870)
@orta
Copy link
Copy Markdown
Contributor Author

orta commented Aug 14, 2020

It might only work on single comments? Will try next time

@jakebailey jakebailey deleted the codeowners branch May 17, 2024 22:01
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