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

Add descriptions and new labels to the bot rules #18342

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

carlos-zamora
Copy link
Member

Wanted to learn how the bot works, so I went ahead cleaned up the bot rules a bit. List of changes:

  • added a description for each rule (and move it to the top of the rule)
  • added all the "Area-" labels and sorted

If there's any new rules you'd like added, let me know. 🎁

@@ -533,14 +609,14 @@
then:
- removeLabel:
label: Needs-Tag-Fix
description:
- if:
- description: Add "In-PR" label to isses that are referenced in a PR

Check failure

Code scanning / check-spelling

Unrecognized Spelling

[isses](#security-tab) is not a recognized word. \(unrecognized-spelling\)
@@ -701,8 +777,8 @@
pattern: Severity-
- labelSync:
pattern: Impact-
description:
- if:
- description: Comments with "/dup", "/dupe", or "/duplicate" targetting another repo will close the issue as a duplicate and remove "Needs-" labels

Check failure

Code scanning / check-spelling

Unrecognized Spelling

[targetting](#security-tab) is not a recognized word. \(unrecognized-spelling\)
Copy link
Member

@DHowett DHowett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

minor spellbot concerns

@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something label Jan 21, 2025
Copy link
Member

@lhecker lhecker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is great. I don't understand what it is with the spellbot.

Copy link
Contributor

This pull request has been automatically marked as stale because it has been marked as requiring author feedback but has not had any activity for 7 days. It will be closed if no further activity occurs within 7 days of this comment.

@microsoft-github-policy-service microsoft-github-policy-service bot added the No-Recent-Activity This issue/PR is going stale and may be auto-closed without further activity. label Jan 31, 2025
@carlos-zamora carlos-zamora removed Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something No-Recent-Activity This issue/PR is going stale and may be auto-closed without further activity. labels Jan 31, 2025
@carlos-zamora
Copy link
Member Author

Well, the spellbot isn't complaining. Now it's just the GitOps/YmlValidation, but I'm having trouble finding anything on it and the error is not helpful. Hoping running the pipeline fixes it 🤷‍♂️. Any guidance is appreciated here.

/azp run

Copy link
Contributor

1 file(s) have code issues.

File Issues
.github/policies/resourceManagement.yml While parsing a block mapping, did not find expected key.

Total execution time: 0.04 seconds

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