-
Notifications
You must be signed in to change notification settings - Fork 409
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
Latest version does not support Java 9 Modules #440
Labels
difficulty: unknown or n/a
fix is unknown in difficulty
status: help wanted
requesting help from the community
type: community enhancement
feature request not on Twilio's roadmap
Comments
thinkingserious
added
type: community enhancement
feature request not on Twilio's roadmap
status: help wanted
requesting help from the community
up-for-grabs
difficulty: unknown or n/a
fix is unknown in difficulty
labels
Jun 25, 2018
Thanks for reporting this @benstpierre! I have added this to our backlog. For this issue to gain priority in our backlog, we need additional +1's or a PR. When we receive a PR, that provides the biggest jump in priority. |
Closed
Since there has been no activity on this issue since March 1, 2020, we are closing this issue. Please feel free to reopen or create a new issue if you still require assistance. Thank you! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
difficulty: unknown or n/a
fix is unknown in difficulty
status: help wanted
requesting help from the community
type: community enhancement
feature request not on Twilio's roadmap
Issue Summary
The latest sendgrid api does not support Java 9 modules. Not only is there no module-info.java your api uses split packages which disqualifies it from use in the java 9 Module system.
You get this error when you try to use it...
Module 'yourmodule' reads package 'com.sendgrid' from both 'sendgrid.java' and 'java.http.client'
Steps to Reproduce
Try to use sendgrid 4.2.1 in a Java 9 or Java 10 app with a module-info.java file.
This article explains.
https://msayag.github.io/SplitPackage/
The text was updated successfully, but these errors were encountered: