Skip to content

docs: name contract/ as Apache-2.0 in LICENSING.md and NOTICE - #2006

Open
frahlg wants to merge 1 commit into
masterfrom
docs/licensing-contract-apache
Open

frahlg wants to merge 1 commit into
masterfrom
docs/licensing-contract-apache

Conversation

@frahlg

@frahlg frahlg commented Nov 12, 2026

Copy link
Copy Markdown
Member

Problem

#1998 moved contract/ to Apache-2.0 on 10 November. contract/LICENSE, contract/README.md and CONTRIBUTING.md say so. LICENSING.md still calls go/protocol "the exception", and NOTICE records only the wire-format module.

Change

  • LICENSING.md lists both Apache-2.0 paths: contract/ and go/protocol.
  • NOTICE gets a dated section for contract/, in the same form as the one for go/protocol.

No code changes, so this PR carries the no-changeset label.

Evidence

scripts/test_package_linux.py, scripts/test_package_legacy_linux.py and scripts/test_history_converter_package.py pass locally (16 tests). The packaging scripts reference these files.

Open question

The go/protocol relicensing waited for a lawyer to confirm it (#1588). The same check may apply to contract/, whose files were AGPL from 12 September to 10 November.

Found while checking license text across the FTW repositories. srcfl/ftw-webapp has a separate fix for its relay and escrow READMEs; neither PR depends on the other.

🤖 Generated with Claude Code

#1998 moved contract/ to Apache-2.0 on 10 November. contract/LICENSE,
contract/README.md and CONTRIBUTING.md say so, but LICENSING.md still
called go/protocol the only exception and NOTICE recorded only the
wire-format module. Both now name the two Apache-2.0 paths.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Signed-off-by: Fredrik Ahlgren <fredrik@sourceful-labs.com>
@frahlg frahlg added the no-changeset PR intentionally exempt from the changeset requirement (dev tooling / non-shipping) label Nov 12, 2026
@chatgpt-codex-connector

Copy link
Copy Markdown
Codex Review Summary: ✅ Completed

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-11-12T07:59:37.569526Z a4f63db PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@miravoss26 miravoss26 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Docs only: LICENSING.md now lists contract/ next to go/protocol as Apache-2.0 exceptions, and NOTICE gets a dated section for contract/ in the same form as the protocol one.

  • No security findings. No code, deps or hosts touched.
  • Text matches what the PR says #1998 did (10 Nov); "Core links it and reads the contracts" is accurate as a non-change to Core's license.
  • One check shows CANCELLED ("changeset present (or auto-exempt)"); everything else is green or skipped. Likely a superseded run, worth a re-run before merge.

Needs a human decision on the open question in the description: #1588 waited for a lawyer on go/protocol, and contract/ was AGPL from 12 Sep to 10 Nov, so the same sign-off may apply before this lands. Otherwise safe to merge from my read.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changeset PR intentionally exempt from the changeset requirement (dev tooling / non-shipping)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants