Skip to content
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

Merged
merged 1 commit into from
Jan 13, 2025
Merged

Conversation

LHT129
Copy link
Collaborator

@LHT129 LHT129 commented Jan 3, 2025

No description provided.

@LHT129 LHT129 added the kind/improvement Code improvements (variable/function renaming, refactoring, etc. ) label Jan 3, 2025
@LHT129 LHT129 self-assigned this Jan 3, 2025
URL_HASH MD5=1d2c7975edba60e995abe3c4af6480e5
DOWNLOAD_NO_PROGRESS 1
INACTIVITY_TIMEOUT 5
TIMEOUT 60md5
Copy link
Collaborator

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?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

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
Copy link
Collaborator

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

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

Copy link

codecov bot commented Jan 3, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Impacted file tree graph

@@            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     

see 11 files with indirect coverage changes

@LHT129 LHT129 force-pushed the yamlcpp branch 2 times, most recently from 6e38b16 to 9eabb0b Compare January 7, 2025 11:39
Copy link
Collaborator

@wxyucs wxyucs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Copy link
Collaborator

@jiaweizone jiaweizone left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@LHT129 LHT129 merged commit 440de18 into antgroup:main Jan 13, 2025
10 of 11 checks passed
@LHT129 LHT129 deleted the yamlcpp branch January 13, 2025 02:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/improvement Code improvements (variable/function renaming, refactoring, etc. ) size/S
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants