Skip to content

Port to SBT 1.2.8, sbt-scala-module, and Scala 2.13.0 #22

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

Merged
merged 1 commit into from
Jul 3, 2019
Merged

Port to SBT 1.2.8, sbt-scala-module, and Scala 2.13.0 #22

merged 1 commit into from
Jul 3, 2019

Conversation

Toromtomtom
Copy link
Contributor

I went ahead and adapted the project as far as I was able to. I'm not very familiar with SBT, so I am afraid that this PR is 60% cargo cult. It might be helpful anyway. I will be happy to make adjustments as necessary if someone guides me a bit.

Fixes #3

@@ -1,11 +1,10 @@
language: scala

jdk:
- oraclejdk8
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why remove that jdk version? In the build.sbt file you tell sbt to use jdk 8 for publishing, so we should keep that version. You can use openjdk8 instead of oraclejdk8, if there is any issue with the latter.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks, that seems to have worked

Copy link
Collaborator

@julienrf julienrf left a comment

Choose a reason for hiding this comment

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

That looks good to me, thanks! Do you mind squashing your commits into a single one?

@Toromtomtom
Copy link
Contributor Author

I just realized that the commit message is a bit sloppy. If necessary I can reword it and push the commit again.

@julienrf
Copy link
Collaborator

julienrf commented Jul 1, 2019

Can someone from the org.scala-lang organization approve or dismiss this PR? I think that in order to effectively publish releases we also need to provide secrets (sonatype credentials) to the Travis CI.

@SethTisue SethTisue merged commit 22bb87a into scala:master Jul 3, 2019
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.

Setup release process
3 participants