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

Dotnet 8 Update #187

Merged
merged 11 commits into from
Nov 17, 2023
Merged

Dotnet 8 Update #187

merged 11 commits into from
Nov 17, 2023

Conversation

sqeezy
Copy link
Collaborator

@sqeezy sqeezy commented Sep 7, 2023

This pull request is intended to add support for dotnet 8. IMO this should only be done after GA of dotnet 8 in November.

How should we organise the work on this? I can supply the new definitions, but can't handle the nuget release.

@jediwhale
Copy link
Owner

I agree we should release when dotnet 8 goes to GA. We could also drop dotnetcore 3.1 since it went out of support last year.

@sqeezy
Copy link
Collaborator Author

sqeezy commented Sep 15, 2023

I think think im through for now. When GA is released we can finalize this PR.

@sqeezy sqeezy marked this pull request as ready for review November 14, 2023 17:37
@sqeezy
Copy link
Collaborator Author

sqeezy commented Nov 14, 2023

@jediwhale
I finished up integration of dotnet 8. It would be great if you could merge and create the new nuget package. We could automate that part of the process for future releases if you like.

Now that I look over the repo once more, we could also add assembly versioning for this release. I think about a file providing the version or writing the version into build.proj instead of Fitsharp.nuspec. What are your thoughts about that?

@jediwhale jediwhale merged commit f24d603 into master Nov 17, 2023
1 check passed
@jediwhale jediwhale deleted the dotnet8 branch November 17, 2023 20:53
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.

2 participants