Skip to content

Skip CLA check for Cursor#7030

Merged
gonzaloriestra merged 1 commit into
mainfrom
skip-cla-for-cursor
Mar 17, 2026
Merged

Skip CLA check for Cursor#7030
gonzaloriestra merged 1 commit into
mainfrom
skip-cla-for-cursor

Conversation

@gonzaloriestra
Copy link
Copy Markdown
Contributor

@gonzaloriestra gonzaloriestra commented Mar 17, 2026

WHY are these changes introduced?

When Cursor is the author of a commit, the CLA check is failing:

Error: In order to merge this pull request, all contributors must sign [Shopify’s CLA](https://cla.shopify.com/).

- @cursoragent: [Sign the CLA](https://cla.shopify.com/) and comment "I have signed the CLA!" to re-run the checks and have your PR reviewed.
CLA: 509e7c51ece228447c2475180442c3b0b856734a Done

WHAT is this pull request doing?

Skips the check for Cursor

How to test your changes?

Merge and try again

Measuring impact

How do we know this change was effective? Please choose one:

  • n/a - this doesn't need measurement, e.g. a linting rule or a bug-fix
  • Existing analytics will cater for this addition
  • PR includes analytics changes to measure impact

Checklist

  • I've considered possible cross-platform impacts (Mac, Linux, Windows)
  • I've considered possible documentation changes

@gonzaloriestra gonzaloriestra force-pushed the skip-cla-for-cursor branch 2 times, most recently from 1b93735 to 076938e Compare March 17, 2026 13:12
Made-with: Cursor
@gonzaloriestra gonzaloriestra marked this pull request as ready for review March 17, 2026 13:17
@gonzaloriestra gonzaloriestra requested a review from a team as a code owner March 17, 2026 13:17
@github-actions
Copy link
Copy Markdown
Contributor

Coverage report

St.
Category Percentage Covered / Total
🟡 Statements 77.31% 14597/18880
🟡 Branches 70.89% 7237/10209
🟡 Functions 76.31% 3708/4859
🟡 Lines 78.8% 13801/17513

Test suite run success

3811 tests passing in 1473 suites.

Report generated by 🧪jest coverage report action from 67f7602

Copy link
Copy Markdown
Contributor Author

gonzaloriestra commented Mar 17, 2026

Merge activity

  • Mar 17, 2:01 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Mar 17, 2:01 PM UTC: @gonzaloriestra added this pull request to the GitHub merge queue with Graphite.

@gonzaloriestra gonzaloriestra added this pull request to the merge queue Mar 17, 2026
Merged via the queue into main with commit 6047afe Mar 17, 2026
22 checks passed
@gonzaloriestra gonzaloriestra deleted the skip-cla-for-cursor branch March 17, 2026 14:11
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