search

fastify / website-metalsmith

This project is used to build the website for fastify web framework and publish it online.
https://www.fastify.io
MIT License
73 stars 132 forks source link

fix typo #402

Closed harshmangalam closed 1 year ago

harshmangalam commented 1 year ago

Checklist

netlify[bot] commented 1 year ago

Deploy Preview for fastify-website ready!

Name Link
Latest commit d3da487f3a0ff0397889d23c11afe710530d8344
Latest deploy log https://app.netlify.com/sites/fastify-website/deploys/6452ab95025f3200086cc4d4
Deploy Preview https://deploy-preview-402--fastify-website.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

luisorbaiceta commented 1 year ago

This is not a typo but a gramatical decision. We are using the past tense but I think it makes sense to consider the present as used in the rest of the document. What do you think @Fdawgs ?

Fdawgs commented 1 year ago

I think the existing use of past tense is fine, the decision to use Pino was made long ago (thus the past tense) and Fastify has stuck by it. Past tense is also used in the next bullet point ('built'), so it is consistent.