-
Notifications
You must be signed in to change notification settings - Fork 39
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
Add support for .NET Trimming #40
base: master
Are you sure you want to change the base?
Conversation
is it worth screwing around with MergeWith or should I just upload the coverage for the latest target (in this PR, net5.0)? |
Latest target is fine by me. The reason I omitted version from folder is so that actions don't need to be updated when target is changed. |
now I just need to figure out what the actual filepaths are 😅 |
You want to add |
405 not allowed? 😕 |
It's broken in master too 🤷 |
that's better |
Release workflow probably needs path update too. Merge whenever you want. |
Shall we target net6 with this instead? It's LTS. |
Sure. I'm thinking it might also be worth redoing this from scratch with the latest linker and analyzers. |
Resolves #39.
Work-in-progress. It looks about right to me but I haven't put it through its paces just yet to see if it (a) still works, and (b) trims the appropriate stuff.