-
Notifications
You must be signed in to change notification settings - Fork 102
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
Add changelog for 5.6.14 #6415
Add changelog for 5.6.14 #6415
Conversation
@sbesson suggested changes included |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor stylistic comments but overall this makes more sense to me.
Are we planning to run a minimal test with an artifact built with these changes to ensure it remains compatible with Java 8?
@sbesson done |
@sbesson any other comments? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No other change suggestions from my side. Just the one question around Java 8 testing from #6415 (review) and whether you need anything here?
For the test, I will have to prepare a setup for that and push a release candidate |
I pushed to my account a tag including the commits proposed in this PR https://github.com/jburel/openmicroscopy/releases/tag/v5.6.14rc1 |
|
Merging to that the history gets included in the doc as part of tomorrow scheduled job |
Entry for 5.6.14