Skip to content

fix(templates): route definition comment line - #575

Closed
cristianossd wants to merge 1 commit into
fastify:mainfrom
cristianossd:cris/fix-routes-comment
Closed

fix(templates): route definition comment line#575
cristianossd wants to merge 1 commit into
fastify:mainfrom
cristianossd:cris/fix-routes-comment

Conversation

@cristianossd

Copy link
Copy Markdown

This patch is minor. It updates the comment line of the routes definition that was using the same as the plugins comment block.

Checklist

Comment thread templates/app-esm/app.js
})

// This loads all plugins defined in routes
// This loads all routes defined in routes

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.

The whole sentence does seem correct: a plugin contains the routes

Maybe we should say "this loads all the plugins defined in routes folder"

@Tony133

Tony133 commented Aug 31, 2026

Copy link
Copy Markdown
Member

I'm closing this PR because it's been open since 2022, and a few years have passed.

If this change is still relevant, feel free to open a new PR based on the current code. 😉 Thanks for your contribution!

@Tony133 Tony133 closed this Aug 31, 2026
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