-
Notifications
You must be signed in to change notification settings - Fork 24
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
introduce third party yaml-cpp for tools #294
Conversation
extern/yaml-cpp/yaml-cpp.cmake
Outdated
URL_HASH MD5=1d2c7975edba60e995abe3c4af6480e5 | ||
DOWNLOAD_NO_PROGRESS 1 | ||
INACTIVITY_TIMEOUT 5 | ||
TIMEOUT 60md5 |
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.
what is the meaning of 60md5?
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.
done
extern/yaml-cpp/yaml-cpp.cmake
Outdated
URL https://github.com/jbeder/yaml-cpp/archive/refs/tags/0.8.0.tar.gz | ||
# this url is maintained by the vsag project, if it's broken, please try | ||
# the latest commit or contact the vsag project | ||
http://vsagcache.oss-rg-china-mainland.aliyuncs.com/yaml-cpp/v3.1.tar.gz |
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.
change v3.1.tar.gz to 0.8.0.tar.gz
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.
done
Codecov ReportAll modified and coverable lines are covered by tests ✅ @@ Coverage Diff @@
## main #294 +/- ##
==========================================
+ Coverage 49.80% 49.98% +0.17%
==========================================
Files 331 331
Lines 32705 32703 -2
Branches 4129 4128 -1
==========================================
+ Hits 16289 16345 +56
+ Misses 15555 15494 -61
- Partials 861 864 +3 |
6e38b16
to
9eabb0b
Compare
Signed-off-by: LHT129 <[email protected]>
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
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
No description provided.