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

Targeting .NET 8 #196

Closed
wants to merge 4 commits into from
Closed

Conversation

brodny
Copy link

@brodny brodny commented Apr 18, 2024

No description provided.

@brodny brodny closed this Apr 18, 2024
@brodny brodny deleted the tech/targeting_net_8 branch April 18, 2024 18:16
@brodny brodny restored the tech/targeting_net_8 branch April 18, 2024 18:17
@fl3pp
Copy link

fl3pp commented Sep 3, 2024

@brodny May I ask why you abandoned this PR? Is it because you didn't receive a response?

@johngit22
Copy link

johngit22 commented Sep 3, 2024 via email

@brodny
Copy link
Author

brodny commented Sep 3, 2024

@fl3pp No, not quite, I was doing it just for my own amusement ;) I created PR by accident before I had a chance to properly test it, and now I cannot dedicate the time it needs because of personal matters. I also had the ambition to enable nullable reference types for the project but same issues as above. Sorry about that but if you need .NET8 support, feel free to take this PR and move on, I won't mind ;)

@johngit22
Copy link

johngit22 commented Sep 3, 2024 via email

@fl3pp
Copy link

fl3pp commented Sep 3, 2024

Sorry for the confusion! I was reading about the issue #194 and wondered why this PR was closed. Seems like I inferred too much here.

Anyway, thanks to both of you for the quick replies

@johngit22
Copy link

johngit22 commented Sep 3, 2024 via email

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