Skip to content
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

Updated to NET8 since NET6 is obsolete #469

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

snakefoot
Copy link
Contributor

No description provided.

@snakefoot snakefoot force-pushed the net8 branch 4 times, most recently from f0b6f25 to 9e6a84c Compare December 10, 2024 15:32
@snakefoot snakefoot force-pushed the net8 branch 2 times, most recently from c9d42a4 to 16dee0c Compare January 7, 2025 18:53
@snakefoot
Copy link
Contributor Author

@Mpdreamz Possible to re-trigger the build one more time?

Mpdreamz
Mpdreamz previously approved these changes Jan 29, 2025
Copy link
Member

@Mpdreamz Mpdreamz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excellent work as always @snakefoot ! Sorry it took a while on my end to review and get this in.

flobernd
flobernd previously approved these changes Jan 29, 2025
Copy link
Member

@flobernd flobernd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Just one Q.

@@ -14,7 +14,9 @@ runs:
# Install .NET version as mandated by global.json
- uses: actions/setup-dotnet@v4
with:
global-json-file: global.json
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What's the reason for this change? Do we need both, net6 and net8 installed?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was needed for nupkg-validator but the bump to 0.5.0 should have fixed this 🤞 will revert this to verify.

Starting with .NET 9.0 the SDK check on dotnet tools is relaxed which is great.

@Mpdreamz Mpdreamz dismissed stale reviews from flobernd and themself via 1ce0778 January 29, 2025 19:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants