Repository navigation
Conversation
The contributing guide asked contributors to sign a Contributor License Agreement via a Google Form, without explaining why the CLA exists, what it covers, where signature records live, or how a signature is verified before a pull request is merged. - docs/guide/contributing.md gains a Contributor License Agreement section: why it is required, that one signature covers both HyperFormula and Handsontable, the four steps a contributor sees, that cla/signed is a required check, where records are stored, and a note telling reviewers the check is the verification. - Replaces the retired Google Form link with the signing page in the contributing guide, the PR template, README.md, and docs/index.md. The signature is recorded per GitHub account rather than per repository, so signing from either project clears the other.
✅ Deploy Preview for hyperformula-dev-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
hyperformula-docs | 96bf2f2 | Commit Preview URL Branch Preview URL |
Aug 11 2026, 09:31 AM |
Performance comparison of head (96bf2f2) vs base (61ead73) |
The gate is now served on its own hostname in the Handsontable Cloudflare account, so the docs no longer reference a sandbox workers.dev URL.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #1725 +/- ##
========================================
Coverage 97.31% 97.31%
========================================
Files 195 195
Lines 15719 15719
Branches 3455 3455
========================================
Hits 15297 15297
Misses 414 414
Partials 8 8 🚀 New features to boost your workflow:
|
| 4. The check turns green — on this PR and on any other open PR of yours, in | ||
| either repository. | ||
|
|
||
| `cla/signed` is a **required check**, so an unsigned PR cannot be merged. |
There was a problem hiding this comment.
Holding my approval on two things, neither of which needs a text change:
cla/signedis not a required check on this repository yet. Measured today: it is missing from thedevelopandmasterrequired contexts (14 entries each), and GitHub reportsisRequired: falsefor it on this PR while every CI context reportstrue. This line (and the "the required check is the verification" note at L80) would be published as false if merged before your "Before merging" item 2. You have admin here, so this is your flip; I'll approve as soon asisRequiredmeasurestrue. Item 1 looks done already: a PR from an existing signer shows "CLA signed … (earlier signing form)".- The branch conflicts with
develop(see the other comment).
The text itself matches what the bot actually does on real external PRs (fail + one comment with the personal link on #1749; green about a minute after the bot's reminder on #1727; org-member exemption on this PR), so nothing else to change.
| ## Contributing | ||
|
|
||
| Contributions are welcome, but before you make them, please read the [Contributing Guide](https://hyperformula.handsontable.com/guide/contributing.html) and accept the [Contributor License Agreement](https://goo.gl/forms/yuutGuN0RjsikVpM2). | ||
| Contributions are welcome, but before you make them, please read the [Contributing Guide](https://hyperformula.handsontable.com/guide/contributing.html) and accept the [Contributor License Agreement](https://cla.handsontable.com/sign). |
There was a problem hiding this comment.
This branch conflicts with develop in this file and in .github/pull_request_template.md (line 24) only; the other two files auto-merge. #1750 (HF-353) changed the Contributing Guide link on the same lines to https://hyperformula.handsontable.com/docs/guide/contributing.html, and #1715 and #1775 also touched README.md since. When resolving, please keep develop's /docs/guide/contributing.html URL and apply only the CLA-link change on top. Develop still has the Google Form link at README.md:110 and template line 24, so the change is still needed.
| `cla/signed` status check. | ||
| 2. If you have not signed, the check fails and a bot comments with your signing | ||
| link. | ||
| 3. Open the link, authenticate with GitHub, read the agreement, and submit the |
There was a problem hiding this comment.
Optional. One external contributor replied /cla sign to the bot on #1749, expecting a comment command as other CLA bots use, and the check stayed red. Either add a sentence here ("There is no comment command — signing happens only on the linked page.") or, cleaner and without diverging from the Handsontable wording, teach the gate worker to answer /cla … comments with the personal link.
This branch has not been deployed
3 participants
Context
Documents the Contributor License Agreement process, which until now pointed contributors at a Google Form with no explanation attached.
The contributing guide asked contributors to sign a CLA, but nothing explained why it is required, what it covers, where signature records live, or how a signature is verified before a pull request is merged. PRO-1312 asks for that explanation in one place, for HyperFormula and Handsontable alike.
Two facts in particular were not written down anywhere a contributor would look:
cla/signedstatus check on every pull request, so an unsigned PR cannot be merged and no reviewer has to check anything by hand.What changed:
docs/guide/contributing.md— newContributor License Agreementsection covering the reasoning, the cross-project scope, the four steps a contributor sees, the required check, where records are stored, and a note for reviewers. Step 1 of "How to get started" now links to it..github/pull_request_template.md,README.md,docs/index.md— the retired Google Form link is replaced by the signing page.The signing page and the full process description are served by the CLA gate worker, which lives in a separate repository. The Handsontable counterpart to this PR is linked below; both repositories deliberately carry the same wording.
How did you test your changes?
Documentation-only change; no library code is touched.
docs/.vuepress/dist/, which is rebuilt):#contributor-license-agreementmatches the new heading, so step 1 of "How to get started" resolves both on GitHub and on the rendered docs site.Types of changes
Related issues:
Checklist:
cla/signedcheck on this PR confirms it.(The OpenDocument/Excel/Sheets and changelog items do not apply to a documentation-only change.)
Before merging
Two operational steps must land first, or contributors get a worse experience than today:
cla/signedas a required check ondevelop, so it actually blocks a merge. The App is now installed here and the check has been verified green on this PR.ClickUp task: https://app.clickup.com/t/9015210959/PRO-1312
Note
Low Risk
Documentation-only changes with no runtime or library code impact.
Overview
Replaces the retired Google Form CLA link with cla.handsontable.com/sign in the PR template,
README.md, anddocs/index.md, and clarifies that one signature covers HyperFormula and Handsontable via thecla/signedcheck.Adds a Contributor License Agreement section to
docs/guide/contributing.md(why the CLA exists, cross-repo scope, PR flow with the GitHub App, required check, EU D1 storage, reviewer guidance). Step 1 of “How to get started” now links to that section instead of the old form URL.Reviewed by Cursor Bugbot for commit 25c96da. Bugbot is set up for automated code reviews on this repo. Configure here.