[docs]: Routing - timeLog not defined

[gregfenton]

Either have a function named timeLog or reword the last sentence of the page to remove the use of that name.

[crandmck]

This looks right to me, but I'd like someone to confirm.

[UlisesGascon]

LGTM! Thanks @gregfenton, this improves the readability 👍

[crandmck]

The tests were failing because two lines of example code ended with colons. By convention, docs don't terminate code lines with colons. I edited the fork/branch directly so we can land this.

[gregfenton]

Interesting that tests were failing because of valid code 😊

@crandmck Sorry for the trouble, and thank you for the fix. If there's a CONTRIBUTIONS set of instructions I missed, apologies for that.

[crandmck]

No worries @gregfenton Thanks for the contribution!
See https://expressjs.com/en/resources/contributing.html#collaborators-guide.
For semicolon usage, https://standardjs.com/rules#semicolons .

[gregfenton]

Cool, thanks for that!

Of course, this means that "fly by edits via GitHub UI" like I did are ... not inline with those standards. My bad.

Cheers.