Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

[Cleanup] CRTP header only#219

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
coder137 merged 7 commits intomainfromcrtp_header_only
Oct 21, 2022
Merged

[Cleanup] CRTP header only#219

coder137 merged 7 commits intomainfromcrtp_header_only
Oct 21, 2022

Conversation

@coder137
Copy link
Owner

Convert CRTP header-class design to CRTP header only wherever possible for TargetInfo

@codecov
Copy link

codecovbot commentedOct 21, 2022

Codecov Report

Base:99.16% // Head:99.17% // Increases project coverage by+0.00% 🎉

Coverage data is based on head(22c4a7c) compared to base(4d9a71e).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@           Coverage Diff           @@##             main     #219   +/-   ##=======================================  Coverage   99.16%   99.17%           =======================================  Files          70       70             Lines        2035     2047   +12       Branches      569      571    +2     =======================================+ Hits         2018     2030   +12  Partials       17       17
Impacted FilesCoverage Δ
buildcc/lib/target/include/target/target_info.h100.00% <ø> (ø)
buildcc/lib/target/src/api/lib_api.cpp100.00% <ø> (ø)
buildcc/lib/target/include/target/api/deps_api.h100.00% <100.00%> (ø)
...uildcc/lib/target/include/target/api/include_api.h100.00% <100.00%> (ø)
buildcc/lib/target/include/target/api/lib_api.h100.00% <100.00%> (ø)
buildcc/lib/target/include/target/api/pch_api.h100.00% <100.00%> (ø)
buildcc/lib/target/include/target/api/source_api.h100.00% <100.00%> (ø)
...ldcc/lib/target/include/target/common/target_env.h100.00% <100.00%> (ø)
buildcc/lib/target/src/api/sync_api.cpp100.00% <100.00%> (ø)
buildcc/lib/target/src/api/target_getter.cpp100.00% <100.00%> (ø)
... and5 more

Help us with your feedback. Take ten seconds to tell ushow you rate us. Have a feature suggestion?Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment?Let us know in this issue.

@coder137coder137 merged commit4f9dd90 intomainOct 21, 2022
@coder137coder137 deleted the crtp_header_only branchOctober 21, 2022 04:25
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@coder137
[8]ページ先頭
©2009-2025 Movatter.jp