From 989080fc868c1df55c4e185b9145493f7f1e6443 Mon Sep 17 00:00:00 2001 From: Tran Duc Anh Dang Date: Sun, 8 Nov 2026 13:17:16 +1100 Subject: [PATCH 1/3] docs: version 0.1 in force, legal review pending --- .github/workflows/cla.yml | 4 ++-- CLA.md | 2 +- README.md | 10 +++++----- REVIEW.md | 2 +- 4 files changed, 9 insertions(+), 9 deletions(-) diff --git a/.github/workflows/cla.yml b/.github/workflows/cla.yml index 3244254..622f933 100644 --- a/.github/workflows/cla.yml +++ b/.github/workflows/cla.yml @@ -1,6 +1,6 @@ -# DRAFT, not in force. A reusable check that the author of a pull request has agreed to the +# A reusable check that the author of a pull request has agreed to the # Orbyn Individual Contributor Licence Agreement. It runs only when a repository calls it -# (on: workflow_call); no repository does yet. See ../../REVIEW.md. +# (on: workflow_call). See ../../REVIEW.md. # # One agreement covers every Orbyn repository: the author comments `/orbynAgree` once, the # signature goes to the single file signatures/icla-.json in orbyn-dev/cla-signatures, diff --git a/CLA.md b/CLA.md index fb1d377..d7a94ad 100644 --- a/CLA.md +++ b/CLA.md @@ -1,6 +1,6 @@ # Orbyn Individual Contributor Licence Agreement -**Version 0.1, DRAFT for legal review. Not in force.** This is not legal advice. Nothing here applies to anyone until it is reviewed, approved and published as version 1.0. +**Version 0.1, in force; legal review pending.** A reviewed version may replace it, and contributors would then be asked to agree again. This is not legal advice. > Notes for counsel are in [the review notes](REVIEW.md). Bracketed text in capitals, such as `[JURISDICTION]`, is a placeholder still to be decided. diff --git a/README.md b/README.md index a6d4cd5..07ae9f5 100644 --- a/README.md +++ b/README.md @@ -1,9 +1,9 @@ # orbyn-dev/.github -> **DRAFT, not in force, pending legal review.** -> Nothing in this repository applies to any Orbyn project yet. No repository uses its workflows, and no contribution is accepted or refused because of it. +> **Version 0.1 of the contributor agreement is in force; legal review is pending.** +> A reviewed version may replace it, and contributors would then be asked to agree again. A repository uses these workflows only after its caller is merged. -This repository will hold organisation-wide files for Orbyn projects, starting with a contributor licence agreement. The draft is under review in a pull request. +This repository holds organisation-wide files for Orbyn projects, starting with the contributor licence agreement. Contact: hello@orbyn.dev @@ -13,9 +13,9 @@ Contact: hello@orbyn.dev Product repositories call it from `examples/licence-grant-caller.yml`. -## Contributor agreement check (draft, not in force) +## Contributor agreement check -`.github/workflows/cla.yml` is a reusable check for the draft [contributor agreement](CLA.md). A contributor comments `/orbynAgree` once on their own pull request; that covers every covered Orbyn repository. Organisation members and bots pass at once. The skills library (`orbyn-fullstack-skills`) stays MIT with a DCO and is not covered. +`.github/workflows/cla.yml` is a reusable check for the [contributor agreement](CLA.md). A contributor comments `/orbynAgree` once on their own pull request; that covers every covered Orbyn repository. Organisation members and bots pass at once. The skills library (`orbyn-fullstack-skills`) stays MIT with a DCO and is not covered. ### Turning it on for a repository diff --git a/REVIEW.md b/REVIEW.md index ae69f27..5a9f3ec 100644 --- a/REVIEW.md +++ b/REVIEW.md @@ -1,6 +1,6 @@ # Review notes for counsel -**DRAFT, not in force. Not legal advice.** These notes explain the draft in [CLA.md](CLA.md) and list what needs a decision before version 1.0. +**Version 0.1 in force; legal review pending. Not legal advice.** These notes explain the agreement in [CLA.md](CLA.md) and list what counsel should decide before a reviewed version replaces it. ## Why a CLA, and why this base From a16ca781bbc8b640ea3f14b1125e4cfd5246fd46 Mon Sep 17 00:00:00 2001 From: Tran Duc Anh Dang Date: Sun, 8 Nov 2026 13:55:30 +1100 Subject: [PATCH 2/3] fix: the comment prefilter used toLower, which Actions does not have; contains() is already case-insensitive --- .github/workflows/cla.yml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/.github/workflows/cla.yml b/.github/workflows/cla.yml index 622f933..11b8862 100644 --- a/.github/workflows/cla.yml +++ b/.github/workflows/cla.yml @@ -56,7 +56,8 @@ jobs: github.event_name == 'pull_request_target' || (github.event_name == 'issue_comment' && github.event.action == 'created' && github.event.issue.pull_request && github.event.comment.user.type != 'Bot' && - contains(toLower(github.event.comment.body), 'agree')) + # contains() ignores case, so "/orbynAgree" and "/ORBYNAGREE" both pass this cheap prefilter. + contains(github.event.comment.body, 'agree')) concurrency: group: orbyn-cla-${{ github.repository }}-${{ github.event.pull_request.number || github.event.issue.number }} cancel-in-progress: false From 2b98a07fa39653c76bb16506d39b1c48444a2b68 Mon Sep 17 00:00:00 2001 From: Tran Duc Anh Dang Date: Sun, 8 Nov 2026 13:55:36 +1100 Subject: [PATCH 3/3] fix: no comment inside the folded condition --- .github/workflows/cla.yml | 1 - 1 file changed, 1 deletion(-) diff --git a/.github/workflows/cla.yml b/.github/workflows/cla.yml index 11b8862..3456a97 100644 --- a/.github/workflows/cla.yml +++ b/.github/workflows/cla.yml @@ -56,7 +56,6 @@ jobs: github.event_name == 'pull_request_target' || (github.event_name == 'issue_comment' && github.event.action == 'created' && github.event.issue.pull_request && github.event.comment.user.type != 'Bot' && - # contains() ignores case, so "/orbynAgree" and "/ORBYNAGREE" both pass this cheap prefilter. contains(github.event.comment.body, 'agree')) concurrency: group: orbyn-cla-${{ github.repository }}-${{ github.event.pull_request.number || github.event.issue.number }}