Skip to content

[19.0][FIX] fastapi: LGPL-3 header on the two files that carried AGPL-3.0 - #657

Merged
OCA-git-bot merged 1 commit into
OCA:19.0from
gonzalesedwin1123:19.0-fix-fastapi-license-headers
Oct 16, 2026
Merged

OCA-git-bot merged 1 commit into
OCA:19.0from
gonzalesedwin1123:19.0-fix-fastapi-license-headers

Conversation

@gonzalesedwin1123

Copy link
Copy Markdown
Contributor

fastapi declares LGPL-3 in its manifest, and every file in the module carries the License LGPL-3.0 or later header, except two: models/ir_rule.py and models/res_lang.py carry License AGPL-3.0 or later, since the commits that added them. The rest of the module, and the manifest, have been LGPL-3 since the first commit, so the two headers look like a copied header template rather than a licence decision.

This changes the two header lines to the one the other files use. No code changes.

If the AGPL header on these two files was intended, please say so here and close this; downstream packagers then know to treat the module as mixed-licence.

The module declares LGPL-3 in its manifest and every other file carries
the LGPL-3 header. models/ir_rule.py and models/res_lang.py carried the
AGPL-3.0 header since they were added, which reads as a copied template
rather than a licence choice; this aligns them with the manifest.
@oca-cla-bot

oca-cla-bot Bot commented Oct 16, 2026 •

Copy link
Copy Markdown

All commit authors signed the Contributor License Agreement.

CLA signed

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hi @lmignon,
some modules you are maintaining are being modified, check this out!

@lmignon

lmignon commented Oct 16, 2026

Copy link
Copy Markdown
Contributor

/ocabot merge patch

@OCA-git-bot

Copy link
Copy Markdown
Contributor

On my way to merge this fine PR!
Prepared branch 19.0-ocabot-merge-pr-657-by-lmignon-bump-patch, awaiting test results.

@OCA-git-bot
OCA-git-bot merged commit d08d473 into OCA:19.0 Oct 16, 2026
8 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at 3457c4f. Thanks a lot for contributing to OCA. ❤️

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants