-
-
Notifications
You must be signed in to change notification settings - Fork 37
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
feat: add Probot-Autolabeler
configuration for documentation
#101
Conversation
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 @biru-codeastromer the format seems incorrect, could you please follow the one in https://github.com/jenkins-infra/jenkins.io/blob/master/.github/autolabeler.yml?
.github/autolabeler.yml
Outdated
@@ -0,0 +1,3 @@ | |||
documentation: | |||
- README.md | |||
- src/user-story/**/*.yaml |
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.
Why are we including this:
stories/.github/autolabeler.yml
Line 3 in 8333c13
- src/user-story/**/*.yaml |
Why should changes to these files be considered changes in documentation?
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.
Sir I thought the .yaml
files under src/user-story/ define structured user stories Therefore, changes to these files will align with documentation updates.
What's your view on this do you think it is valid ? Let me know if it's not required here .
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.
The src/user-story/**/*.yaml
contain the actual contents of the featured stories, not documentation
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.
Got the clarification thanks. Sure , I am removing now .
Probot-Autolabeler
configuration for documentation
I have removed the |
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
Thanks ! |
Fixes #100
Probot-Autolabeler
bot to automate some PR labelings #100Description :
Key Details:
yaml
files .