Skip to content

gha: add bootstrap_repo workflow & task for incubator CI - #133

Merged
cognifloyd merged 40 commits into
masterfrom
gha
May 19, 2022
Merged

cognifloyd merged 40 commits into
masterfrom
gha

Conversation

@cognifloyd

Copy link
Copy Markdown
Member

we need to let all with write access merge PRs.
restricting pushes to TSC also prevents non-TSC from merging PRs.

also, we need to remove admin access from the bot at the end.
@CLAassistant

CLAassistant commented May 11, 2022 •

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@cognifloyd
cognifloyd marked this pull request as ready for review May 19, 2022 06:31
@cognifloyd

Copy link
Copy Markdown
Member Author

Hooray! This is ready for review. See details here: StackStorm-Exchange/exchange-incubator#172 (comment)

@arm4b arm4b left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The amount of dedication definitely feels here.
Looks really nice & clean 👍

"dismissal_restrictions": {},
"dismiss_stale_reviews": false,
"require_code_owner_reviews": true,
"required_approving_review_count": 1,

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👍

# So, we patch it separately.
run: >
gh api -X PUT '/orgs/{owner}/teams/${{ inputs.tsc_team }}/repos/{owner}/{repo}'
-f permission=maintain

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👍

pack_repo_template:
type: string
required: false
default: StackStorm-Exchange/ci-pack-template

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👍

gha is actually case-insensitive here, so this is purely a cosmetic change.
@cognifloyd
cognifloyd merged commit 1043015 into master May 19, 2022
@cognifloyd

Copy link
Copy Markdown
Member Author

Thanks for the review @armab !

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