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

Update Groovy versions #11800

Merged
merged 1 commit into from Oct 27, 2021
Merged

Update Groovy versions #11800

merged 1 commit into from Oct 27, 2021

Conversation

chrisvest
Copy link
Contributor

Motivation:
The old Groovy versions are running into compatibility issues with the latest JDK snapshots.

Modification:
Update the groovy maven plugin and its groovy dependency to the latest versions.
It looks like groovy-all is now a pom-dependency, so we switch to just groovy since we don't need what's in -all beyond that.

Result:
We now build on the latest JDK snapshots.

Motivation:
The old Groovy versions are running into compatibility issues with the latest JDK snapshots.

Modification:
Update the groovy maven plugin and its groovy dependency to the latest versions.
It looks like groovy-all is now a pom-dependency, so we switch to just groovy since we don't need what's in -all beyond that.

Result:
We now build on the latest JDK snapshots.
@chrisvest chrisvest added this to the 5.0.0.Final milestone Oct 26, 2021
@normanmaurer normanmaurer merged commit 6aa5b15 into netty:main Oct 27, 2021
@chrisvest chrisvest deleted the 5x-update-groovy branch October 27, 2021 11:22
normanmaurer pushed a commit that referenced this pull request Oct 27, 2021
Motivation:
The old Groovy versions are running into compatibility issues with the latest JDK snapshots.

Modification:
Update the groovy maven plugin and its groovy dependency to the latest versions.
It looks like groovy-all is now a pom-dependency, so we switch to just groovy since we don't need what's in -all beyond that.

Result:
We now build on the latest JDK snapshots.
laosijikaichele pushed a commit to laosijikaichele/netty that referenced this pull request Dec 16, 2021
Motivation:
The old Groovy versions are running into compatibility issues with the latest JDK snapshots.

Modification:
Update the groovy maven plugin and its groovy dependency to the latest versions.
It looks like groovy-all is now a pom-dependency, so we switch to just groovy since we don't need what's in -all beyond that.

Result:
We now build on the latest JDK snapshots.
laosijikaichele pushed a commit to laosijikaichele/netty that referenced this pull request Dec 16, 2021
Motivation:
The old Groovy versions are running into compatibility issues with the latest JDK snapshots.

Modification:
Update the groovy maven plugin and its groovy dependency to the latest versions.
It looks like groovy-all is now a pom-dependency, so we switch to just groovy since we don't need what's in -all beyond that.

Result:
We now build on the latest JDK snapshots.
raidyue pushed a commit to raidyue/netty that referenced this pull request Jul 8, 2022
Motivation:
The old Groovy versions are running into compatibility issues with the latest JDK snapshots.

Modification:
Update the groovy maven plugin and its groovy dependency to the latest versions.
It looks like groovy-all is now a pom-dependency, so we switch to just groovy since we don't need what's in -all beyond that.

Result:
We now build on the latest JDK snapshots.
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.

None yet

2 participants