A few improvements#30
Open
selul wants to merge 6 commits into
Open
Conversation
* improve issue string parsing * add output for issues linking state * fix connected issue workflow
adds no_comment arg
|
Thanks for this @selul and @hattan, these features look really helpful! Do you know if there are any plans for a new release with these changes merged soon? I'm especially interested in detecting the linked issues in the sidebar (#22). It would be really great to have functionality that checks only the sidebar and ignores the comments. This would be so we can check that each PR has a primary linked issue for quality management purposes. |
|
This project looks dead I wonder if we should fork this project and keep it up to date with things like this and ignoring Dependabot PR's etc?? |
wesProg23
approved these changes
Feb 14, 2023
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hi there,
I have played recently with this action and I have found a few things where I could help.
issue-parserpackage, which now detects cross-repo issues ( full URL ), ie.https://github.com/hattan/verify-linked-issue-action/issues/29as well as shorthand ones,#issue_numberforEachnot inside the main function, possible connected to add ability to detect issue referenced in sidebar #22quietwhich prevent action for failing, allowing you to use the output in other stepsno_commentwhich prevent the action from adding the comment.quietargument automatically for this as true.Let me know if you find those useful.
P.S one example where I used the output is this: