Skip to content

Conversation

@dondonz
Copy link
Member

@dondonz dondonz commented Jun 12, 2022

This PR shades antlr-runtime and removes it off the pom.

antlr-runtime version clashes can be a blocker as described in #2809 and openrewrite/rewrite#1615

}

/**
* @return map of coerced variables
Copy link
Member Author

Choose a reason for hiding this comment

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

Fixing up javadoc warnings, unrelated to antlr

@jbellenger
Copy link
Contributor

Very excited to see this! The antlr runtime has prevented us from using graphql-java in systems that use incompatible antlr versions.

@bbakerman bbakerman added this to the 19.0 milestone Jun 14, 2022
@bbakerman bbakerman merged commit f21869a into master Jun 14, 2022
@dondonz dondonz deleted the shade-ANTLR-runtime branch June 14, 2022 06:07
@jord1e
Copy link
Contributor

jord1e commented Jul 27, 2022

Just noting that #2789 will allow running ANTLR 4.10.x generation with JDK11 whilst preserving JDK8 source compatibilty
I will merge the current featureset into #2789 again today.

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