-
Notifications
You must be signed in to change notification settings - Fork 1
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
#205: Release process is broken #206
Conversation
* removed incompatible settings from sbt project setup * separated Sonatype and GitHub release, as they use different Java versions * updated build.yml for newest action versions
JaCoCo model module code coverage report - scala 2.13.11
|
JaCoCo agent module code coverage report - scala 2.13.11
|
JaCoCo server module code coverage report - scala 2.13.11
|
|
||
val clientJavacOptions: Seq[String] = Seq("-source", "1.8", "-target", "1.8", "-Xlint") | ||
def clientScalacOptions(scalaVersion: Version): Seq[String] = { | ||
if (scalaVersion >= scala213) { | ||
Seq("-release", "8", "-Ymacro-annotations") | ||
} else { | ||
Seq("-target", "8", "-release", "8") | ||
Seq("-release", "8", "-target:8") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is it really needed together? "-target:8"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🤷
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM although I asked 2 questions - otherwise I built it locally and it works
Co-authored-by: Ladislav Sulak <[email protected]>
Closes #205