Skip to content

Exclude jersey-constants from the release test#6069

Open
arjantijms wants to merge 1 commit intoeclipse-ee4j:4.xfrom
arjantijms:4.x
Open

Exclude jersey-constants from the release test#6069
arjantijms wants to merge 1 commit intoeclipse-ee4j:4.xfrom
arjantijms:4.x

Conversation

@arjantijms
Copy link
Copy Markdown
Contributor

From 2.x branch

Signed-off-by: Maxim Nesen <senivam@gmail.com>
@arjantijms arjantijms requested review from dmatej and jansupol February 2, 2026 13:30
@dmatej
Copy link
Copy Markdown
Contributor

dmatej commented Feb 2, 2026

Could you add description why is that, please?

@mkarg
Copy link
Copy Markdown
Member

mkarg commented Feb 25, 2026

Could you add description why is that, please?

@arjantijms Yes please, as it makes it easier for others to review and confirm.

@senivam
Copy link
Copy Markdown
Contributor

senivam commented Mar 20, 2026

I'd say it's only propagating the exclusion from the 2.x branch.

Regarding why it's required, the release test is developed to check modules after build (before release to Sonatype), if they are correctly assembled, if all multireleases are really multirelease jars, if OSGi headers are correct, if there are LICENSE and NOTICE files in place, if NOTICE files contain proper versions vs. the real version of dependencies. In this relation, the constants module is very simple and should be excluded from the OSGi header check because its OSGi header is not required and is not present. That is why it's being excluded.

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.

4 participants