search

microsoft / PowerApps-Tooling

Tooling support for PowerApps language and .msapp files
https://make.powerapps.com
MIT License
322 stars 107 forks source link

Upgrade to .net 8 #657

Closed jack-work closed 4 months ago

jack-work commented 4 months ago

If this is related to an issue open in GitHub, please link it to this ticket and put the URL here.

Problem

.Net 7 is at end of life and we need to migrate to 8 to remain in compliance. My ultimate intention is to upgrade document server, among this packages primary dependents, to .Net 8. https://msazure.visualstudio.com/OneAgile/_workitems/edit/28159300/

This required me to change many files to fix code analysis errors. Some of the errors may be controversial. I am open to any suggestions and recommendations on what to and what not to suppress.

petrochuk commented 4 months ago

Can we separate .net 8 upgrade from code analysis fixes?

jack-work commented 4 months ago

Andrew did this by suppressing all of the analyzer errors. If we want to take those up, this pr might save some time, but I don't mind if we abandon it.