Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update README.md #381

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update README.md #381

wants to merge 1 commit into from

Conversation

cwolter97
Copy link

Updated README.md to show Fastify version bump

Checklist

Updated README.md to show Fastify version bump

Signed-off-by: Cody Wolter <cwolter97@gmail.com>
@Eomm Eomm linked an issue Oct 25, 2024 that may be closed by this pull request
2 tasks
@@ -15,8 +15,8 @@ This plugin can be used in a variety of circumstances, for example if you have t

## Requirements

Fastify 4.x.
See [@fastify/http-proxy v7.x](https://github.com/fastify/fastify-http-proxy/tree/v7.x) for Fastify 3.x compatibility.
Fastify 5.x.
Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Author

Choose a reason for hiding this comment

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

I don't disagree, I just wanted to follow the existing format to stay within scope of the issue I created. Rather than start to reformat how the compatibility is shown.

But I do like this approach, rather than clicking (potentially several times) to find the plugin version that's compatible with a specific Fastify version.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update README.md to state support for Fastify 5.x
2 participants