Migrate license to FSL-1.1-ALv2#508
Conversation
📝 WalkthroughWalkthroughThe repository licensing model is changed from Apache License 2.0 to Functional Source License 1.1 (ALv2 Future License). The original Apache 2.0 license file is removed, replaced with the FSL 1.1 license terms, and README documentation is updated to reflect the new licensing designation for Liquibase 4. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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 |
|
Claude finished @filipelautert's task in 44s —— View job Review Complete
One factual error found in The PR changes this line: to: This is incorrect. The FSL-1.1-ALv2 license text (in All other changes look correct — the LICENSE file swap, README table and section updates for Liquibase 4's license, and the new LICENSE.txt content match the canonical FSL-1.1-ALv2 text. |
Co-authored-by: claude[bot] <209825114+claude[bot]@users.noreply.github.com>
|
Claude finished @filipelautert's task in 1m 48s —— View job Review Complete
The previously flagged error (" One remaining concern in All other changes look correct — the |
|
Claude finished @filipelautert's task in 1m 30s —— View job Review Complete
One inconsistency found: The previous issue (incorrect "transitions to FSL-1.1-ALv2" instead of "Apache 2.0") was correctly fixed. However, the latest commit (
|
| ### Step 1: Understand License Requirements | ||
|
|
||
| - **Liquibase 4**: Uses Apache 2.0 license (always available) | ||
| - **Liquibase 4**: Uses Apache v2 license (always available) |
There was a problem hiding this comment.
This line still says "Apache v2" but the rest of the document (table, section heading at line 213, section body at line 215, tag descriptions) all say "FSL-1.1-ALv2" for Liquibase 4 — this is an internal inconsistency introduced by the latest commit.
| - **Liquibase 4**: Uses Apache v2 license (always available) | |
| - **Liquibase 4**: Uses FSL-1.1-ALv2 license (always available) |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@README.md`:
- Line 121: Update the conflicting Liquibase 4 license statement so the README
is consistent: choose the correct license identifier (either "FSL-1.1-ALv2" or
"Apache v2") and apply it uniformly where Liquibase 4 is referenced (replace the
text at the current bullet that reads "**Liquibase 4**: Uses Apache v2 license
(always available)" to match the other occurrences or vice versa), and run a
quick search in the README for all occurrences of "Liquibase 4", "FSL-1.1-ALv2",
and "Apache v2" to ensure every mention (including the instances referenced
around lines 19, 213/215/387/400) uses the chosen, identical license phrase.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 700e8c4d-5d2c-4a0a-b9d2-416f365dea5d
📒 Files selected for processing (3)
LICENSELICENSE.txtREADME.md
💤 Files with no reviewable changes (1)
- LICENSE
|
Claude finished @filipelautert's task in 1m 4s —— View job Review Complete
LGTM — the previous issues have been fully resolved.
No significant issues found. |
Summary
This aligns with the license migration completed in the main
liquibase/liquibaserepository in September 2025.Reference