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

Add .NET 8 target framework and use string interpolation #25

Closed
wants to merge 1 commit into from

Conversation

gumbarros
Copy link

No description provided.

@nojaf
Copy link
Contributor

nojaf commented Feb 28, 2024

Hi @gumbarros, thank you for your interest in this project!
Do you have any specific use case in mind to use net8.0 inside this library?
If not, adding that net8.0 doesn't really do much when your own app is targeting the net8.0 runtime.

@gumbarros
Copy link
Author

Good morning!

Nah, just added .NET 8 to see if some new analyzer suggest something you can remove if you want :)

@nojaf
Copy link
Contributor

nojaf commented Feb 28, 2024

That is an interesting sentence there. What did you expect to see from what kind of analyzer potentially?

@gumbarros
Copy link
Author

gumbarros commented Feb 28, 2024

That is an interesting sentence there. What did you expect to see from what kind of analyzer potentially?

At least in C#, I found in my projects the following analyzers:
https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1859
https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1860

I don't have the info of new analyzers for F#

@nojaf
Copy link
Contributor

nojaf commented Feb 28, 2024

Analyzers are not a thing in F#. It will potentially be on the roadmap for F# 9.

@gumbarros gumbarros closed this May 18, 2024
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