-
Notifications
You must be signed in to change notification settings - Fork 46
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
Upgraded to .NET 8 #68
Conversation
Fixes The configured user limit (128) on the number of inotify instances has been reached
hey @gigi81 and thanks for opening this PR! I'll test it later today. Looks like PR build is failing while branch build is green but I guess it will be resolved after merge (do I need to do any setup for test reports though?). Is there anything else you're planning on including in this PR? |
Hey. Not sure what went wrong as I tried to build on my fork and it was fine. Let me try fix the build before merging. Will have a look tomorrow evening |
I pulled the latest version and it had dotnet target framework missing. I fixed that and tested the app locally. After that the build failed on UI tests because xunit execution setting is different now. I fixed that too. Apparently after the upgrade UI tests were disabled, had to enable them back and fix the issue with one of them. Builds are green now. Thanks for the contribution once again, I'm merging this PR |
Upgraded the whole solution to .NET 8
Updated actions to use .NET 8 sdk
Updated to use central package management
Updated readme for SDK 8
Updated dependencies to more recent versions (excluding Avalonia)