Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix the GTest target name case #541

Merged
merged 1 commit into from
Jan 18, 2024
Merged

Conversation

trxcllnt
Copy link
Contributor

The target name is GTest::gtest not GTest::GTest

Copy link

codecov bot commented Jan 18, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (5ef4972) 84.46% compared to head (e68c5f6) 84.38%.

Additional details and impacted files
@@             Coverage Diff              @@
##           mainline     #541      +/-   ##
============================================
- Coverage     84.46%   84.38%   -0.09%     
============================================
  Files            70       70              
  Lines          6063     6063              
  Branches        510      510              
============================================
- Hits           5121     5116       -5     
- Misses          942      947       +5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@hcho3 hcho3 merged commit f0de3cb into dmlc:mainline Jan 18, 2024
17 of 18 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants