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 explicitly the statement that the usage of Giraffe.EndpointRouting is recommended #556

Merged
merged 3 commits into from
Feb 25, 2024

Conversation

64J0
Copy link
Member

@64J0 64J0 commented Sep 1, 2023

Description

With this PR I'm removing some empty spaces from the documentation and adding explicitly the statement that the usage of Giraffe.EndpointRouting is recommended.

How to test

Related issues

Copy link
Member

@TheAngryByrd TheAngryByrd left a comment

Choose a reason for hiding this comment

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

🎸 Thank you!

@TheAngryByrd
Copy link
Member

@dustinmoris I somehow lost being able to merge again

@64J0
Copy link
Member Author

64J0 commented Sep 5, 2023

Up

@TheAngryByrd
Copy link
Member

@dustinmoris I somehow lost being able to merge again

@64J0 64J0 linked an issue Feb 25, 2024 that may be closed by this pull request
@64J0 64J0 force-pushed the add-endpoint-routing-best-practice branch from a3bd168 to 3014da5 Compare February 25, 2024 20:13
@64J0 64J0 merged commit ca8d369 into giraffe-fsharp:develop Feb 25, 2024
5 checks passed
@64J0 64J0 deleted the add-endpoint-routing-best-practice branch February 25, 2024 20:18
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.

Is Giraffe.EndpointRouting considered best practice?
2 participants