-
Notifications
You must be signed in to change notification settings - Fork 155
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
Update and rename _apps to pr-triage.md #194
Conversation
The maintainers of this repository would appreciate it if you could provide more information. |
CI fails but I think it has nothing to do with my change. Any idea ?
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @sotayamashita! Thanks for the PR! You're right in that the build breaking is not your fault cough @JasonEtco cough.
I like your app and the graphic a lot! The file you added does need to go in the /_apps/
folder before this can be merged.
pr-triage.md
Outdated
@@ -0,0 +1,47 @@ | |||
--- | |||
# A human-friendly name of your listing |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All these comments need to be removed for the formatting to work correctly.
Update pr-triage.md pr-triage.md -> _apps/pr-triage.md
@hiimbex I appreciate your quick review. I fixed some issue. Please review it |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
pr-triage/app#31