docs: replace dummy with placeholder in example comments#7064
docs: replace dummy with placeholder in example comments#7064bjohansebas merged 2 commits intoexpressjs:masterfrom
Conversation
Use more inclusive language in code comments across example files. The term 'dummy' can be replaced with the clearer, more neutral term 'placeholder' without any change in meaning. Files changed: - examples/ejs/index.js - examples/route-middleware/index.js - examples/auth/index.js
|
the CI issue is not related to this (503 code error in the coverage upload service), I can retry it again in few hours (ref): |
|
Thanks for the context, @UlisesGascon — just tried pushing a re-trigger commit and hit the same coveralls.io 503. Will leave it with you to retry once the service recovers. |
|
@UlisesGascon Quick nudge — any chance the coverage job can be re-run or bypassed for merge? All tests pass across the full Node matrix, it's just the coveralls 503 holding things up. |
|
Hi @UlisesGascon — the PR has two approvals and all Node matrix tests pass. The only failing check is the Coveralls coverage step which is hitting a 503 from the Coveralls service (external infrastructure issue, not related to this change). Would you be able to re-trigger that check or merge despite it? Happy to wait if there's a preferred process for this. |
|
Please don’t tag people manually 🙂 |
4 participants
Summary
Files changed
Context
The Inclusive Naming Initiative lists dummy as a Tier 3 term to replace when possible. Placeholder is a more precise and neutral alternative that better describes the purpose of the test data in these examples.
Test plan