-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
[#11383] Add detection of PRs linking to closed issues #13154
[#11383] Add detection of PRs linking to closed issues #13154
Conversation
Update message Rephrase message Add logging Test Update pr.yml
…m component tests (TEAMMATES#13148) Update pr.yml Update pr.yml Update pr.yml
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.
Thanks for working on this, but I think we may need to make some changes
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
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, interesting solution!
Part of #11383
Outline of Solution
Modified
pr.yml
to check for the status of the issue that newly created pull requests are solving using thepr.data.state.description
attribute from GitHub's REST API for issues, allowing users to be notified that the issue they are trying to work on is already closed, as per Automation Hero in the wiki. Tested on my personal fork.