Skip to content

Conversation

@eladb
Copy link
Contributor

@eladb eladb commented Mar 4, 2021

I believe the reason our auto-approve workflow does not work is because it was triggered when the pull request was created and before the pr/auto-approve label was applied to it.

This adds types: [ labeled ] so the workflow is triggered when a PR is labeled. The condition remains the same.


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

I believe the reason our auto-approve workflow does not work is because it was triggered when the pull request was _created_ and before the `pr/auto-approve` label was applied to it.

This adds `types: [ labeled ]` so the workflow is triggered when a PR is labeled. The condition remains the same.
@gitpod-io
Copy link

gitpod-io bot commented Mar 4, 2021

@eladb eladb requested a review from nija-at March 4, 2021 12:54
@mergify mergify bot added the contribution/core This is a PR that came from AWS. label Mar 4, 2021
@nija-at nija-at added the pr/do-not-merge This PR should not be merged at this time. label Mar 4, 2021
@eladb
Copy link
Contributor Author

eladb commented Mar 4, 2021

@nija-at thanks, done.

@aws-cdk-automation
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: AutoBuildProject89A8053A-LhjRyN9kxr8o
  • Commit ID: 6f88534
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@eladb eladb merged commit 396acee into master Mar 4, 2021
@eladb eladb deleted the benisrae/fix-auto-approve-take2 branch March 4, 2021 14:00
cornerwings pushed a commit to cornerwings/aws-cdk that referenced this pull request Mar 8, 2021
I believe the reason our auto-approve workflow does not work is because it was triggered when the pull request was _created_ and before the `pr/auto-approve` label was applied to it.

This adds `types: [ labeled ]` so the workflow is triggered when a PR is labeled. The condition remains the same.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contribution/core This is a PR that came from AWS. pr/do-not-merge This PR should not be merged at this time.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants