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

[WIP] tag PRs using alert blocks #107

Open
wants to merge 5 commits into
base: master
from
Open

[WIP] tag PRs using alert blocks #107

wants to merge 5 commits into from

Conversation

@znck
Copy link
Member

znck commented May 25, 2020

Fixes #102

/cc @bencodezen can you decide what label we want to use?

znck added 5 commits May 25, 2020
@znck znck changed the title chore: tag PRs using alert blocks [WIP] tag PRs using alert blocks May 25, 2020
@bencodezen
Copy link
Member

bencodezen commented Jun 8, 2020

@znck Thanks for this PR! As far as your question, I'm thinking it will useful to scope out these labels in case we have other flags we want to check for in the future in docs. So I'm proposing we prefix these flags with review-:

So for this one:

review-alert-box

And then if we want to check for code samples in the future, it could be:

review-code-example

What do you think?

@bencodezen
Copy link
Member

bencodezen commented Jul 15, 2020

@znck Just wanted to check in and see if you needed me to do anything else for this PR. Let me know whenever you have a chance. No rush. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

2 participants
You can’t perform that action at this time.