Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

More release checks #5664

Merged
merged 1 commit into from
Jun 18, 2024
Merged

More release checks #5664

merged 1 commit into from
Jun 18, 2024

Conversation

jansupol
Copy link
Contributor

No description provided.

Signed-off-by: jansupol <jan.supol@oracle.com>
final int minVersion;
if (jerseyVersion.startsWith("4")) {
minVersion = 17;
} else if (jerseyVersion.startsWith("3.1")) {
Copy link
Member

@jbescos jbescos May 31, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In case we create a new 3.2 version we will need to remember to update this or it will fall to version 8. Maybe it is preferable to use a regular expression, something like: ^3\.[1-9]+ (please test it because I am not good with regular expressions).

@senivam senivam merged commit e24eb6d into eclipse-ee4j:2.x Jun 18, 2024
7 checks passed
@senivam senivam added this to the 2.44 milestone Jun 18, 2024
@jansupol jansupol deleted the release.test.1 branch June 20, 2024 10:05
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants