Skip to content
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

Workflow for Approval Bot Dispatch #20376

Merged
merged 1 commit into from
Dec 12, 2022
Merged

Conversation

git-phu
Copy link
Contributor

@git-phu git-phu commented Dec 12, 2022

What

Add ability to use auto approve/merge bot in oss repo.

How

Repository dispatch to the auto approve/merge bot workflow defined in https://github.com/airbytehq/airbyte-cloud/pull/3764

@git-phu git-phu marked this pull request as ready for review December 12, 2022 17:20
@git-phu git-phu requested a review from a team December 12, 2022 17:21
Copy link
Contributor

@supertopher supertopher left a comment

Choose a reason for hiding this comment

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

Does this limit the bot to working only with Airbyte people?

id: scd
with:
token: ${{ env.PAT }}
permission: write
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@supertopher this should limit who can invoke the bot (need to have at least write permission on this repo)

Copy link
Contributor

Choose a reason for hiding this comment

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

ack. ty

@git-phu git-phu temporarily deployed to more-secrets December 12, 2022 18:03 — with GitHub Actions Inactive
@git-phu git-phu temporarily deployed to more-secrets December 12, 2022 18:04 — with GitHub Actions Inactive
@git-phu git-phu merged commit 3c8bb42 into master Dec 12, 2022
@git-phu git-phu deleted the peter/approval-bot-dispatch branch December 12, 2022 18:27
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