Skip to content

ARCH-1192 - Switch to pr_target & add descriptions for use #3

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

Merged
merged 2 commits into from
Mar 21, 2022

Conversation

danielle-casella-adams
Copy link
Contributor

Summary of PR changes

  • In the increment-version-on-merge workflow switch to using the pull_request_target so when it is merged it will have the correct permissions to create and push a new version tag on the repo.
  • The descriptions of pull_request/pull_request_target have been updated so reviewers should know which scenarios each should be used in the workflows.
  • The version of the action has not been updated on the readme's since this change won't affect the usage.
    • When I have some time to implement the readme version update on merge all of the versions should be reconciled.

@danielle-casella-adams danielle-casella-adams requested review from a team as code owners March 18, 2022 23:03
@danielle-casella-adams danielle-casella-adams merged commit edae3c8 into main Mar 21, 2022
@danielle-casella-adams danielle-casella-adams deleted the pr-target branch March 21, 2022 14:28
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