-
Notifications
You must be signed in to change notification settings - Fork 575
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
fix(ci): sqlsmith test filepath #8183
Conversation
I've submitted an empty commit while debugging the buildkite service, please ignore that. Thanks :) |
Large diff due to pre-generated sqlsmith test files. We have 100 of them now. |
If we increase past that, will move these into a separate repository / zip the files? |
Codecov Report
@@ Coverage Diff @@
## main #8183 +/- ##
==========================================
- Coverage 71.85% 71.84% -0.01%
==========================================
Files 1129 1129
Lines 181263 181263
==========================================
- Hits 130239 130224 -15
- Misses 51024 51039 +15
Flags with carried forward coverage won't be shown. Click here to find out more.
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
That will be better, i.e. doing both I guess. If it’s only useful for CI, not even need to be public |
I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
main-cron
.cron
(since its test num is 100).Checklist For Contributors
./risedev check
(or alias,./risedev c
)Checklist For Reviewers
Documentation
Click here for Documentation
Types of user-facing changes
Please keep the types that apply to your changes, and remove the others.
Release note