Align docs with Apache release distribution#165
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 8568d86f68
ℹ️ 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".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| test('public docs do not contain stale BSL license claims', () => { | ||
| for (const [file, text] of Object.entries(allContents)) { | ||
| assert.doesNotMatch(text, /open[- ]source/iu, `${file} uses open-source for current BSL release`); | ||
| assert.doesNotMatch(text, /production use allowed|allows production use/iu, `${file} overstates BSL production rights`); | ||
| assert.doesNotMatch(text, /BSL|Business Source|Change Date|Additional Use Grant/iu, `${file} contains stale BSL-era license language`); | ||
| assert.doesNotMatch(text, /becomes Apache-2\.0|Apache-2\.0 in 2030|2030-04-20/iu, `${file} contains stale delayed-Apache language`); | ||
| } |
There was a problem hiding this comment.
Include citation metadata in stale-license guard
Because allContents only covers README/SECURITY/.llm/ROADMAP plus launch collateral, this new stale-BSL check still lets public repository metadata through; CITATION.cff:10 currently says license: BSL-1.1, so citation tools and package/index consumers can continue seeing the old license while CI passes. Add CITATION.cff (and any other root public metadata) to the scanned file set so the new Apache-2.0 migration is actually enforced.
Useful? React with 👍 / 👎.
Summary
Verification
rg -n "source-available|Source-available|Source-Available|BSL|Business Source|Change Date|commercial license|Additional Use Grant|MIT license|2030-04-20|Apache-2\.0 in 2030|becomes Apache|Becomes Apache|packages are not published yet|Package publishing is not enabled yet|local build \(packages not published yet\)|npx -y @dialectos|npm install -g @dialectos|v0\.1\.1|0\.1\.1|command\": \"npx\"|open-source \(Apache-2\.0 in 2030\)|free for most use, Apache-2\.0|Apache-2\.0 and Apache-2\.0|Apache-2\.0\. Apache-2\.0" README.md SECURITY.md ROADMAP.md .llm docs packages .github --glob "!**/dist/**" --glob "!docs/plans/**"only reports the intentional guard regexes indocs/__tests__/public-claims.test.mjsnode --test docs/__tests__/*.test.mjs scripts/__tests__/release-workflow.test.mjsNeed help on this PR? Tag
/codesmithwith what you need. Autofix is disabled.