-
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
Feature/#63 create build matrix and build action #64
Feature/#63 create build matrix and build action #64
Conversation
I am missing Jacoco GH Action. Support for Jacoco itself has been add, but GH Action workflow is missing. It should be added together. |
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.
- pulled
- built
- test ran
- code review
- jacoco reports generated
- Jacoco optional report comment generated by GHAction - Missing
project/Dependencies.scala
Outdated
Seq( | ||
"org.apache.spark" %% "spark-core" % sparkVersion % Provided, | ||
"org.apache.spark" %% "spark-sql" % sparkVersion % Provided, | ||
"com.typesafe" % "config" % typesafeVersion, //TODO to check if used |
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.
Found TODO in this file.
In the code there are 4 another TODOs - these should be updated with Issue where is planned to solved them.
Co-authored-by: miroslavpojer <[email protected]>
Co-authored-by: miroslavpojer <[email protected]>
…and-build-action' into feature/#63-Create-build-matrix-and-build-action
JaCoCo server module code coverage report - scala 2.12.12
|
JaCoCo agent module code coverage report - spark:2 - scala 2.12.12
|
JaCoCo server module code coverage report - scala 2.12.12
|
…and-build-action' into feature/#63-Create-build-matrix-and-build-action
JaCoCo agent module code coverage report - spark:2 - scala 2.12.12
|
JaCoCo server module code coverage report - scala 2.12.12
|
JaCoCo agent module code coverage report - spark:2 - scala 2.12.12
|
JaCoCo server module code coverage report - scala 2.12.12
|
JaCoCo agent module code coverage report - spark:2 - scala 2.12.12
|
JaCoCo server module code coverage report - scala 2.12.12
|
) | ||
.enablePlugins(TomcatPlugin) | ||
.enablePlugins(AutomateHeaderPlugin) | ||
.jvmPlatform(scalaVersions = Seq(scala212)) |
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.
won't scala 2.11 be supported?
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.
It will be supported but again, 2.12 should be the default version.
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.
Good for now. 2.13 should be added eventually too.
JaCoCo agent module code coverage report - spark:2 - scala 2.12.12
|
JaCoCo server module code coverage report - scala 2.12.12
|
@@ -0,0 +1,23 @@ | |||
# Release |
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.
Please add the reference to this file into README.md
like we have it in other libs
) | ||
.enablePlugins(TomcatPlugin) | ||
.enablePlugins(AutomateHeaderPlugin) | ||
.jvmPlatform(scalaVersions = Seq(scala212)) |
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.
Good for now. 2.13 should be added eventually too.
project/plugins.sbt
Outdated
addSbtPlugin("za.co.absa.sbt" % "sbt-jacoco" % "3.4.1-absa.4" from "https://github.com/AbsaOSS/sbt-jacoco/releases/download/3.4.1-absa.4/sbt-jacoco-3.4.1-absa.4.jar") | ||
addSbtPlugin("com.thoughtworks.sbt-api-mappings" % "sbt-api-mappings" % "3.0.2") | ||
|
||
// To add release plugin |
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.
Please add this above the JaCoco plugins block. It's kind of lost here.
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.
Nice, about the extra part of code coverage 😎
Notes
Closes #63