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

Upgrade to .NET 9 #49

Merged
merged 6 commits into from
Mar 4, 2025
Merged

Upgrade to .NET 9 #49

merged 6 commits into from
Mar 4, 2025

Conversation

BNAndras
Copy link
Member

I borrowed the updated package versions from the C# test runner. The NuGet audit flag was needed since no internet access means the audit couldn't access NuGet's API.

@BNAndras BNAndras added the x:rep/tiny Tiny amount of reputation label Feb 19, 2025
@BNAndras BNAndras requested a review from a team as a code owner February 19, 2025 06:10
Copy link

Hello 👋 Thanks for your PR.

This repo does not currently have dedicated maintainers. Our guardians team will attempt to review and merge your PR, but it will likely take longer for your PR to be reviewed.

If you enjoy contributing to Exercism and have a track-record of doing so successfully, you might like to become an Exercism maintainer for this track.

Please feel free to ask any questions, or chat to us about anything to do with this PR or the reviewing process on the Exercism forum.

(cc @exercism/guardians)

@BNAndras
Copy link
Member Author

Before this gets merged in, I should update the track exercises and merge everything together. I'm expecting it should be pretty straighforward.

Copy link
Member

@ErikSchierboom ErikSchierboom left a comment

Choose a reason for hiding this comment

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

The NugetAudit thing is weird. I can look into fixing this.

@BNAndras BNAndras force-pushed the bump-dotnet-version branch from e4afc9d to fea65e5 Compare February 26, 2025 06:04
@ErikSchierboom
Copy link
Member

@BNAndras I've fixed the remaining issues. Let me know when you want to merge.

@BNAndras
Copy link
Member Author

BNAndras commented Mar 3, 2025

Thanks! I just need to sync the update PR vbproj files in the main track repo, and then I can merge in everything later today.

@BNAndras BNAndras merged commit d325133 into exercism:main Mar 4, 2025
1 check passed
@BNAndras BNAndras deleted the bump-dotnet-version branch March 4, 2025 05:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
x:rep/tiny Tiny amount of reputation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants