Skip to content

Conversation

@sensslen
Copy link
Contributor

@sensslen sensslen commented Feb 2, 2026

aDD MULTI-TARGETING SUPPORT

@codecov
Copy link

codecov bot commented Feb 3, 2026

Codecov Report

❌ Patch coverage is 71.42857% with 10 lines in your changes missing coverage. Please review.
✅ Project coverage is 79.72%. Comparing base (713b42e) to head (6aadb1e).

Files with missing lines Patch % Lines
src/CommandLineUtils/Internal/ReflectionHelper.cs 16.66% 3 Missing and 2 partials ⚠️
...LineUtils/Conventions/OptionAttributeConvention.cs 80.00% 1 Missing and 2 partials ⚠️
src/CommandLineUtils/CommandLineApplication.cs 50.00% 0 Missing and 1 partial ⚠️
src/CommandLineUtils/Utilities/DotNetExe.cs 0.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #591      +/-   ##
==========================================
- Coverage   79.93%   79.72%   -0.22%     
==========================================
  Files         120      121       +1     
  Lines        4017     4024       +7     
  Branches      872      872              
==========================================
- Hits         3211     3208       -3     
- Misses        618      628      +10     
  Partials      188      188              

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@sensslen sensslen changed the title Almost make multitarget possible Make multitarget possible Feb 3, 2026
@sensslen
Copy link
Contributor Author

sensslen commented Feb 3, 2026

@natemcmaster this PR now almost builds successfully. The only missing bit is that the strong naming does not properly work for .net framework - hence the tests fail. Since I have no knowledge on strong naming whatsoever, can you please help me fix this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant