-
Notifications
You must be signed in to change notification settings - Fork 0
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
#101: ErrorHandling documentation and fields renames #103
Changes from 22 commits
7178f32
de99046
2db8c95
51b16f4
4783fca
c8c4aa0
8e747e7
64dc95e
e29098d
8fe1b27
9543c94
8822c8a
eec8d4f
d9a2129
1c59acf
5b317d8
96c2308
30f50c3
38b54b4
93a5c3b
712931b
b99d06a
efda9fb
6e24db0
68fc0aa
963787c
656dfc5
b789cac
378f1fe
49855db
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -36,6 +36,13 @@ lazy val commonSettings = Seq( | |
Test / parallelExecution := false | ||
) | ||
|
||
lazy val commonJacocoExcludes: Seq[String] = Seq( | ||
"za.co.absa.spark.commons.adapters.CallUdfAdapter", | ||
"za.co.absa.spark.commons.adapters.TransformAdapter" | ||
// "za.co.absa.spark.commons.utils.JsonUtils*", // class and related objects | ||
// "za.co.absa.spark.commons.utils.ExplodeTools" // class only | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. And here as well. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The example with * is still hint. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I will actually change it to make it obvious it's a documentation and not dead code 😉 |
||
) | ||
|
||
lazy val parent = (project in file(".")) | ||
.aggregate(sparkCommons.projectRefs ++ sparkCommonsTest.projectRefs: _*) | ||
.settings( | ||
|
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.
I see dead code, is reserved for something or can removed
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.
Still here (and 1 below)