-
Notifications
You must be signed in to change notification settings - Fork 321
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
build: support build on no tag #3641
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3641 +/- ##
=============================================
- Coverage 74.94% 40.16% -34.79%
Complexity 640 640
=============================================
Files 724 191 -533
Lines 130414 11233 -119181
Branches 1325 1325
=============================================
- Hits 97743 4512 -93231
+ Misses 32378 6428 -25950
Partials 293 293 ☔ View full report in Codecov by Sentry. |
SDK Test Report100 files ±0 100 suites ±0 2m 3s ⏱️ -1s Results for commit fd418aa. ± Comparison against base commit 71cf61e. This pull request removes 30 and adds 9 tests. Note that renamed tests count towards both.
♻️ This comment has been updated with latest results. |
Linux Test Report 52 files - 2 59 suites - 55 27m 9s ⏱️ - 22m 22s For more details on these failures, see this check. Results for commit 5dbfa78. ± Comparison against base commit 71cf61e. This pull request removes 6246 tests.
♻️ This comment has been updated with latest results. |
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
if REF set empty, branch or commit, failed on grep HEAD. Ignore this error, just use snapshot to be the version. The doc is correct, no update about it.
zetasql lib built by workflow run is wierd, but can't solve it now. So we add zetasql lib url to replace it in workflow.