Skip to content

Don't reprompt if there's already been a review on the current dependencies - #19

Open
goldenapples wants to merge 1 commit into
add-plugin-security-reviewfrom
dont-reprompt-for-the-same-hash
Open

Don't reprompt if there's already been a review on the current dependencies#19
goldenapples wants to merge 1 commit into
add-plugin-security-reviewfrom
dont-reprompt-for-the-same-hash

Conversation

@goldenapples

Copy link
Copy Markdown

Built on #18 :

This action shouldn't be prompting multiple times for the same plugin changes. If a review has been dismissed or is already open, then it's not necessary to notify the user again on every commit.

…encies

This action shouldn't be prompting multiple times for the same plugin
changes. If a review has been dismissed or is already open, then it's
not necessary to notify the user again on every commit.

@kadamwhite kadamwhite left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for this.

We'll need to follow up with another commit to bump the SHA references in the docs, but low priority since there's not an internal dependency chain.

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.

2 participants