Skip to content

CLA Action branch change#2847

Merged
brob merged 3 commits intomasterfrom
cla
Mar 27, 2026
Merged

CLA Action branch change#2847
brob merged 3 commits intomasterfrom
cla

Conversation

@brob
Copy link
Copy Markdown
Contributor

@brob brob commented Mar 26, 2026

CLA Action can't go against master branch since it's protected, we'll need to keep a dedicated CLA branch

Summary by CodeRabbit

  • Chores
    • Updated workflow configuration for the continuous integration pipeline.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 26, 2026

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 26, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 6ba6a647-7266-46a3-a997-0655878b4d4a

📥 Commits

Reviewing files that changed from the base of the PR and between 21d25f0 and e4fdee5.

📒 Files selected for processing (1)
  • .github/workflows/cla.yml
✅ Files skipped from review due to trivial changes (1)
  • .github/workflows/cla.yml

📝 Walkthrough

Walkthrough

The CLA GitHub Action workflow configuration is updated to use the 'cla' branch instead of 'master' for the action source reference. This is a single-line parameter modification with no changes to workflow logic or other configurations.

Changes

Cohort / File(s) Summary
GitHub Actions Configuration
.github/workflows/cla.yml
Updated the branch input parameter for fusionauth/cla-github-action from 'master' to 'cla'.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

  • mooreds
  • omer-topal

Poem

🐰 A branch does change, from master's old ways,
To 'cla' the action now displays,
One line adjusted with gentle care,
The workflow flows with branches fair! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: updating the CLA GitHub Action's target branch from 'master' to 'cla', which is the primary modification in the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch cla

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 26, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.63%. Comparing base (7bba8ca) to head (e4fdee5).
⚠️ Report is 69 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2847      +/-   ##
==========================================
- Coverage   82.95%   82.63%   -0.31%     
==========================================
  Files          74       74              
  Lines        8500     8126     -374     
==========================================
- Hits         7050     6714     -336     
+ Misses        913      893      -20     
+ Partials      537      519      -18     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@brob
Copy link
Copy Markdown
Contributor Author

brob commented Mar 26, 2026

I have read the CLA Document and I hereby sign the CLA

@brob brob changed the title Adds CLA callout in Readme CLA Action can't go against master branch since it's protected, we'll need to keep a CLA branch Mar 26, 2026
@brob brob requested a review from mooreds March 26, 2026 20:24
@brob brob changed the title CLA Action can't go against master branch since it's protected, we'll need to keep a CLA branch CLA Action branch change Mar 26, 2026
@brob brob merged commit f860452 into master Mar 27, 2026
11 of 19 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Mar 27, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants