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

Migrate off probot-CLA to new GitHub Action #424

Merged
merged 1 commit into from
Jul 26, 2022

Conversation

cursedcoder
Copy link
Contributor

Hello 👋

We are deprecating probot usage, so all Shopify repos should be migrated to use new GitHub CLA Action.

You can see how this works in vscode-shopify-ruby repo.

If you have any questions, please reach us on Slack.

After you merge this PR, make sure to update branch protection settings (if you have any):

  1. Go to Settings > Branches > Edit main (or any default) branch
  2. Remove the CLA check from probot (click x): image
  3. Add CLA check from GitHub action: image

** In order for the cheak to appear on the list, the workflow should be ran for at least 1 time, so after you merge this PR, add comment with text signed, then refresh the settings page.

How to trigger workflow on existing PRs:

There are few ways how to start the check on PRs created before this change:

a) Close and reopen PR:
image
b) Push empty commit to trigger the events: git commit --allow-empty -m "wakey wakey GitHub Actions"

@cursedcoder cursedcoder requested a review from a team as a code owner July 26, 2022 13:11
@sambostock sambostock merged commit 1f8a974 into main Jul 26, 2022
@sambostock sambostock deleted the replace-cla-probot-with-action branch July 26, 2022 13:55
@sambostock
Copy link
Contributor

signed

@sambostock
Copy link
Contributor

Looks like we didn't have any check requirements, but I've added the cla check requirement.

@shopify-shipit shopify-shipit bot temporarily deployed to rubygems August 5, 2022 19:56 Inactive
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