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

Update to .net 7 #27

Open
wants to merge 11 commits into
base: develop
Choose a base branch
from
Open

Update to .net 7 #27

wants to merge 11 commits into from

Conversation

Trigun27
Copy link

@Trigun27 Trigun27 commented Aug 25, 2023

Thanks to @mlaily, i just updated PR to .net 7

Summary of the changes (also see commit list):

  • Updated to .NET 7.0 and Giraffe 6

  • Extract the views to a C# project so we get full VS support in .cshtml files

    • This is probably the most controversial change of this PR. It seems Visual Studio does not completely support cshtml files in fsproj project, and the intellisense does not work. This considerably reduces the usefulness of Razor views.

    • It complexifies the sample a bit, but this seems like a more realistic usage and I think the tradeoff is worth it.

  • Other minor fixes and improvements.

@Trigun27 Trigun27 changed the title Update Update to .net 7 Aug 25, 2023
@Trigun27
Copy link
Author

@TheAngryByrd hi!
Can you review PR?
Thanks

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.

None yet

3 participants