Skip to content

Update docs - #240

Open
segevfiner wants to merge 4 commits into
fastify:mainfrom
segevfiner:update-docs
Open

Update docs#240
segevfiner wants to merge 4 commits into
fastify:mainfrom
segevfiner:update-docs

Conversation

@segevfiner

Copy link
Copy Markdown

This might also apply to the similar @fastify/express

Checklist

This might also apply to the similar @fastify/express
Signed-off-by: Segev Finer <segev208@gmail.com>
Comment thread README.md

// Catch all routes so that Fastify will call the middlewares above
fastify.all('/css', () => {
return reply

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

reply is not declared

Signed-off-by: Segev Finer <segev208@gmail.com>
Comment thread README.md Outdated
Co-authored-by: Manuel Spigolon <behemoth89@gmail.com>
Signed-off-by: Segev Finer <segev208@gmail.com>
@Tony133
Tony133 requested a review from Eomm June 23, 2026 07:33
Comment thread README.md

The syntax style is the same as [express](http://npm.im/express)/[connect](https://www.npmjs.com/package/connect).
The syntax style is the same as [connect](https://www.npmjs.com/package/connect).
But it does not provide full compatbility with [express](http://npm.im/express) which has its own extended request/response objects, see [@fastify/express](https://github.com/fastify/fastify-express) if that is needed.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Let's use the full path instead of npm.im

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