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
Bot refactoring for easier use and updating #20805
Conversation
no resolution label when anyone comments
IssueManagement.ResolveStale.yml
no resolution label if anyone comments while in 7 day window
IssueManagement.CloseResolutions.yml
"Resolution-No Activity" label and reply message
PRManagement.yml
IssueManagement.ResolveStale.yml
@StevenBucher98 You need to resolve the conflicts. |
This PR has Quantification details
Why proper sizing of changes matters
Optimal pull request sizes drive a better predictable PR flow as they strike a
What can I do to optimize my changes
How to interpret the change counts in git diff output
Was this comment helpful? 👍 :ok_hand: :thumbsdown: (Email) |
📣 Hey @StevenBucher98, how did we do? We would love to hear your feedback with the link below! 🗣️ 🔗 aka.ms/PSRepoFeedback |
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.
I started to review this but due to other things GitHub lost my review, which actually may be for the best as was much longer than this one will be & let us at least get this in and then can be discussed about further adaptions & improvements.
In short @StevenBucher98 I think this PR was good & needed, especially the splitting up of things into smaller files, though perhaps some language in messages and the time frames for closures could be improved to make the experience seem friendlier & account for both the issue/discussion/PR author/collaborators/maintainers having time off like weekends, non-working days, illness & holidays being or collaborators/maintainers unsure as to where to initially triage to as the team doesn't (I don't think) currently or intend to provide a 24x7x365 support experience, but should detail via a response that links to a support policy so that when new issues are raised people can read and get an instant understanding of what should come next & if needs what escalation processes are in place, both via being community/team maintainers/collaborators.
I think we pretty much have almost all of this in place already, but I think but it may just need to connect some missing dots and add an auto response on issue/PR/discussion creation that can link to either a doc or blog post detailing how the team expects to be deal with interactions going forward to better formalise things and make expectations clear, this can then be used as a starting blueprint for interaction across all repos/services of the PowerShell team and I hope should make experiences better for all involved going forward & probs better to see if we can try find some time to discuss more in the new year.
PR Summary
Trying to update the github resource management bot so we can see the behaviors more clearly and segmented out, WIP as I mainly want the bot itself to check for syntax issues
PR Context
PR Checklist
.h
,.cpp
,.cs
,.ps1
and.psm1
files have the correct copyright headerWIP:
or[ WIP ]
to the beginning of the title (theWIP
bot will keep its status check atPending
while the prefix is present) and remove the prefix when the PR is ready.(which runs in a different PS Host).