Conversation
There was a problem hiding this comment.
this Dockerfile does not need update
|
@QiChenX pls make framhash and rapidjson as submoudle in contrib directory. And duing cmake configure, you should check if the target submodule directory is empty, is not, run |
SDK Test Report101 files ±0 101 suites ±0 2m 12s ⏱️ +10s Results for commit 2b03632. ± Comparison against base commit 47a61cb. This pull request removes 30 and adds 9 tests. Note that renamed tests count towards both. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3773 +/- ##
=========================================
Coverage 40.26% 40.26%
Complexity 658 658
=========================================
Files 194 194
Lines 11426 11426
Branches 1378 1378
=========================================
Hits 4601 4601
Misses 6521 6521
Partials 304 304 ☔ View full report in Codecov by Sentry. |
What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
feature
What is the current behavior? (You can also link to an open issue here)
docker image for developers to build openmldb from scratch without internet for native codes #3760
bundle rapidjson & farmhash directly in source code
What is the new behavior (if this is a feature change)?
bundle rapidjson & farmhash directly in source code, allowing users to compile OpenMLDB offline