-
Notifications
You must be signed in to change notification settings - Fork 1.5k
MINOR: Post release of 1.16.0 #3305
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
Conversation
- Set previous version to 1.15.2 for japicmp - Remove optional profile from release script
421e67b to
aeeb55b
Compare
|
LGTM thanks for the change |
|
I was testing this here https://github.com/apache/parquet-java/pull/3302/files, but seems duplicated will close the other one |
| new_development_version="$3-SNAPSHOT" | ||
|
|
||
| git tag -am "Release Apache Parquet $release_version" "$release_tag" "$rc_tag" | ||
| ./mvnw --batch-mode -Pvector-plugins release:update-versions -DdevelopmentVersion="$new_development_version" |
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.
Why this change, shouldn't that profile be part of the release?
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, we haven't officially released it yet and it required Java 17 support. Perhaps we can include it once your proposal on the dev ML to bump to Java 17 has reached consensus?
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.
Ah, got it. Yes, that would be nice. Less maven profiles is more :)
- Set previous version to 1.16.0 for japicmp - Remove optional profile from release script
Rationale for this change
This is the post release process of 1.16.0.
What changes are included in this PR?
Are these changes tested?
No
Are there any user-facing changes?
No