-
Notifications
You must be signed in to change notification settings - Fork 355
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
[CELEBORN-850][INFRA] Add SBT CI #1771
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1771 +/- ##
==========================================
+ Coverage 46.78% 46.96% +0.18%
==========================================
Files 162 162
Lines 10012 10012
Branches 923 923
==========================================
+ Hits 4683 4701 +18
+ Misses 5019 5005 -14
+ Partials 310 306 -4 see 2 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
add flink 1.15/1.17 support add flink-it/flink-client-shaded support rebase branch pure-sbt align final jar name with maven add prefix and update project name enabled ci bump junit jdk 17 try set java.io.tmpdir Revert "try" This reverts commit 9e78f0d. try try disable parallelExecution try upload log try try try debug info print debug info try fix remove celeborn-client/test fail-fast: false test by group fix
@@ -553,7 +573,7 @@ trait SparkClientProjects { | |||
) ++ commonUnitTestDependencies | |||
) | |||
} | |||
|
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.
should it be covered by spotless?
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.
after experimentation, it seems that the maven spotless plugin is unable to format source files located outside the source directory.
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.
not a big deal.
push: | ||
branches: | ||
- main | ||
- branch-* |
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.
Emm ... I'm afraid we should exclude branch-0.3 and lower
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.
Since we don't backport this to branch-0.3, I think it's safe.
What changes were proposed in this pull request?
This PR adds new GitHub Actions workflows to enable Continuous Integration using SBT based on #1764
Does this PR introduce any user-facing change?
No
How was this patch tested?
Pass GA