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

fix: Replace bcprov-jdk15to18 with bcprov-jdk15on #643

Merged
merged 1 commit into from
Jul 28, 2020
Merged

fix: Replace bcprov-jdk15to18 with bcprov-jdk15on #643

merged 1 commit into from
Jul 28, 2020

Conversation

Stephan202
Copy link
Contributor

By upgrading the maven-shade-plugin the bcprov-jdk15on artifact (which contains JDK 9+ classes) can also be used when building the project using JDK 8. This change reverts d9f4413.

Resolves #636.

  • I acknowledge that all my contributions will be made under the project's license.

By upgrading the `maven-shade-plugin` the `bcprov-jdk15on` artifact
(which contains JDK 9+ classes) can also be used when building the
project using JDK 8. This change reverts
d9f4413.

Resolves #636.
@thinkingserious thinkingserious added the status: code review request requesting a community code review or review from Twilio label Jul 28, 2020
@childish-sambino childish-sambino changed the title Replace bcprov-jdk15to18 with bcprov-jdk15on fix: Replace bcprov-jdk15to18 with bcprov-jdk15on Jul 28, 2020
@childish-sambino childish-sambino merged commit 7f5545e into sendgrid:main Jul 28, 2020
@Stephan202 Stephan202 deleted the improvement/issue-636 branch July 28, 2020 19:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: code review request requesting a community code review or review from Twilio
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Question: why the switch from bcprov-jdk15on to bcprov-jdk15to18?
4 participants