Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

fast track: Update main with missing commits #1225

Merged
merged 13 commits into from
Feb 28, 2024

Conversation

jamezp
Copy link
Contributor

@jamezp jamezp commented Feb 21, 2024

This PR is comprised of several commits that are in the release-3.2 and release-4.0 branches that should likely be in master. It also takes the liberty of change the master branches version from 3.1.0 to 4.0.0-SNAPSHOT. This was primarily done to prepare for a slimmed down 4.0.0 for Jakarta EE 11 as described in https://www.eclipse.org/lists/rest-dev/msg00231.html

Associated PR's that comprised this PR:

Just for a reference too, this PR includes the following commits from release-4.0:

@spericas
Copy link
Contributor

So if master becomes the new 4.0 for EE11, isn't it confusing to have a 3.2 branch? I know that it is experimental, but seems a bit odd for its number to be lower. Maybe I'm missing something.

@jamezp
Copy link
Contributor Author

jamezp commented Feb 22, 2024

So if master becomes the new 4.0 for EE11, isn't it confusing to have a 3.2 branch? I know that it is experimental, but seems a bit odd for its number to be lower. Maybe I'm missing something.

TBH I completely agree. I'm just not sure what to do. The master branch is out of date. The release-4.0 branch can't really become master unless we revert some things like removing the the @Context and @Suspended annotations along with their respective documentation. The release-4.0 branch also has the TCK module commented out so it doesn't compile it and really it can't because of the removed annotations.

Both master, release-3.2 and release-4.0 include #1066 so we need some kind of version bump in master. If we remove the XML Binding dependency, then it needs to be a major bump.

In the end this seemed like the best middle ground. There really should be another commit on top of this, which I can add, that updates the signature test file too because it is missing the singantures from #1066 and the signature change from #863.

IMO none of the branches are in a releasable state. I will go ahead and add the commit to update the signature files to this PR too. I think that is critical too. Whether we go with this PR or not is fine, but IMO we do need some branch in a releasable state.

@jamezp
Copy link
Contributor Author

jamezp commented Feb 22, 2024

I've added the additional commit to update the signature test plugin and file.

Please note this is just my opinion on what would get the branch where we need it. I wanted to propose it so we could have a path forward. Given, AIUI, this would take at least two weeks to get merged, that leaves with about two more weeks to get anything else done for Jakarta EE 11. We're due to have something by 29.Mar.2024, unless the schedule gets pushed.

@jim-krueger jim-krueger changed the title Update master with missing commits fast track: Update master with missing commits Feb 23, 2024
@jim-krueger
Copy link
Contributor

Given, AIUI, this would take at least two weeks to get merged, that leaves with about two more weeks to get anything else done for Jakarta EE 11. We're due to have something by 29.Mar.2024, unless the schedule gets pushed.

I've added "fast track" to the title. Since the changes in this PR have all been reviewed and approved in other PRs it seems logical to allow this to be fast tracked.

@jim-krueger
Copy link
Contributor

So if master becomes the new 4.0 for EE11, isn't it confusing to have a 3.2 branch?

I can rename the "release-3.2" branch to "release-4.1" to remove confusion. If for some reason the EE11 schedule gets pushed out and the group feels comfortable with the content in the release-4.1 branch, I'm assuming we could apply it to 4.0.

@jim-krueger jim-krueger changed the title fast track: Update master with missing commits fast track: Update main with missing commits Feb 23, 2024
pom.xml Show resolved Hide resolved
@jamezp
Copy link
Contributor Author

jamezp commented Feb 26, 2024

For ease of review I've done the copyright date updates in a separate commit. If preferred, I could do the updates within the correct commit the file was changed in.

jim-krueger
jim-krueger previously approved these changes Feb 26, 2024
@jamezp jamezp force-pushed the update-master branch 2 times, most recently from 2da825a to d553f79 Compare February 26, 2024 20:18
@jim-krueger
Copy link
Contributor

Merging

@jim-krueger jim-krueger merged commit a725e56 into jakartaee:main Feb 28, 2024
4 checks passed
@jamezp jamezp deleted the update-master branch March 1, 2024 16:46
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.

5 participants