#56816 closed defect (bug) (fixed)
Correct references to npm and Node.js
| Reported by: | desrosj | Owned by: | desrosj |
|---|---|---|---|
| Priority: | normal | Milestone: | 6.2 |
| Component: | General | Version: | |
| Severity: | normal | Keywords: | good-first-bug has-patch |
| Cc: | Focuses: | docs |
Description
The correct names are npm and Node.js. They're spelled/capitalized differently in various areas. Let's correct these.
- instances of
NPMwithin GitHub Action workflows, Core markdown files, and a few translatable strings/inline comments. - 14 instances of
NodeJSwithin GitHub Action workflows.
Change History (26)
This ticket was mentioned in PR #3472 on WordPress/wordpress-develop by @aleks7391.
4 years ago
#1
- Keywords has-patch added; needs-patch removed
#2
@
4 years ago
Heyo, I replaced all the wrong references of npm and Node.js with the correct ones. Hope all is good.
#3
@
4 years ago
- Owner set to
- Status new → reviewing
Thanks @aleksganev! Now that 6.1 has been branched, this can be committed any time.
I'll review this at some point in the coming days.
#6
@
4 years ago
It looks like props were missed in the commit, I have corrected that manually in the Core Props tool on make/core.
Note:
See TracTickets
for help on using tickets.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Replaced all instances of "NPM" and "NodeJS" with "npm" and "Node.js" respectively, which are the correct names
Trac ticket: https://core.trac.wordpress.org/ticket/56816