Skip to content

Update LICENSE/NOTICE for 1.4.0 release#3864

Open
snazy wants to merge 4 commits intoapache:mainfrom
snazy:re-add-deps
Open

Update LICENSE/NOTICE for 1.4.0 release#3864
snazy wants to merge 4 commits intoapache:mainfrom
snazy:re-add-deps

Conversation

@snazy
Copy link
Member

@snazy snazy commented Feb 23, 2026

This change effectively prepares the LICENSE files for the 1.4.0 release.

  • Requires the changes from Fix LICENSE and NOTICE for admin tool and server (prerequisite) #3881
  • Re-adds the license checks removed a while ago. This verifies the dependencies of the effective dependencies in the Polaris Admin tool & Polaris Server Quarkus applications by ensuring that the dependencies are mentioned in the LICENSE file.
  • The Polaris distribution assembly's LICENSE & NOTICE files are now generated from the LICENSE & NOTICE file contents of Polaris Admin tool & Polaris Server. In other words: do not need to be maintained separately.

Fixes #2484

@snazy
Copy link
Member Author

snazy commented Feb 23, 2026

This change includes / supersedes #3861 (discussed offline w/ @jbonofre)

@snazy
Copy link
Member Author

snazy commented Feb 23, 2026

Depends on #3865 and #3866

@snazy snazy force-pushed the re-add-deps branch 4 times, most recently from 80ea9f9 to dad3664 Compare February 24, 2026 10:21
@snazy snazy force-pushed the re-add-deps branch 3 times, most recently from 93aaf02 to f5dd459 Compare February 24, 2026 17:52
@snazy snazy marked this pull request as ready for review February 24, 2026 17:53
@snazy
Copy link
Member Author

snazy commented Feb 24, 2026

I know this is a pretty big change, but given the time constraints, I opted to put the whole thing into one PR.

If reviewers prefer, I can split this into a few PRs.

@adnanhemani
Copy link
Contributor

@snazy I see you've commented that these comments are "part of #3881" - but none of those comments are fixed on that PR. Will copy over the comments.

To reduce confusion, maybe you can remove the license changes from this PR in preparation of #3881 merging?

@snazy snazy force-pushed the re-add-deps branch 2 times, most recently from 2625a71 to f6f160c Compare February 26, 2026 09:42
... from LICENSE/NOTICE files from admin + server
@snazy snazy marked this pull request as ready for review March 3, 2026 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Simplify distribution LICENSE/NOTICE

2 participants