Skip to content

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

Merged
childish-sambino merged 1 commit into
sendgrid:mainfrom
PicnicSupermarket:improvement/issue-636
Jul 28, 2020
Merged

fix: Replace bcprov-jdk15to18 with bcprov-jdk15on#643
childish-sambino merged 1 commit into
sendgrid:mainfrom
PicnicSupermarket:improvement/issue-636

Conversation

@Stephan202

Copy link
Copy Markdown
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