-
Notifications
You must be signed in to change notification settings - Fork 124
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 IL trimming support for DotNext.Metaprogramming #218
Conversation
What do you think about adding an empty console app project to the solution that just serves the purpose of testing for trimming errors in all the |
Sounds reasonable. |
Ok, I pushed the addition of that project to my fork: alexrp@af824ce As expected,
Any of these you'd prefer to tackle? If not, I'll take a look over the next few days. |
There are few warnings related to |
Feel free to modify a branch linked to this PR directly. |
Can I actually do that? 👀 I tried pushing the commit to your feature branch here first, but GitHub rejected that due to permissions. I don't know if there's some other way to do it. 🤔 |
Perhaps, you never contributed to this repository and didn't pass CLA check. I'll close this PR without merge. Could you please open a new one with your branch and merge changes from this branch? |
Yep, will do. |
Addresses #217
Scope of work:
DynamicDependency
andDynamicallyAccessedMembers
where applicableRequiresUnreferencedCode
otherwise