Closed thompson-tomo closed 5 months ago
I added some comments per-file, but other than that I get the spirit of your PR and I agree: less explicit dependencies for TFMs that do not require them, great!
In case you agree about the changes, let me know if you want to make them yourself or I can do them.
Thanks!
Happy to make changes @jodydonetti but atm I can't see any proposed changes. Is there a chance you are yet to click finish on your review which publishes the comments?
Happy to make changes @jodydonetti but atm I can't see any proposed changes. Is there a chance you are yet to click finish on your review which publishes the comments?
Yep, I forgot to submit the review, sorry about that 🤦
No problem. Review Comments have just been published so will look at them shortly & come back/make changes.
Add in net coreapp 3.1 & net 6 as TFM to a number of projects so that conditions can be placed on the dependencies to removed explicit dependencies on the newer frameworks
Closes #230