Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: refer to npm consistently (branding) #812

Merged
merged 4 commits into from
Mar 29, 2024

Conversation

MikeMcC399
Copy link
Collaborator

Issue

This repo refers to npm - a JavaScript package manager inconsistently using both upper case "NPM" and lower case "npm" in workflow-related comments and documentation.

Usage of capitalized "NPM" is however not in line with npm branding guidelines which state:

npm should never be capitalized unless it is being displayed in a location that is customarily all-capitals (ex. titles on man pages).

Changes

Change all relevant usage of "NPM" to lower case "npm", aligning with npm branding guidelines.

@cypress-app-bot
Copy link
Collaborator

@MikeMcC399

This comment was marked as outdated.

@MikeMcC399

This comment was marked as outdated.

@MikeMcC399 MikeMcC399 marked this pull request as ready for review March 25, 2024 10:41
@MikeMcC399

This comment was marked as resolved.

@MikeMcC399 MikeMcC399 marked this pull request as draft March 25, 2024 10:42
@MikeMcC399 MikeMcC399 marked this pull request as ready for review March 27, 2024 14:45
@jennifer-shehane jennifer-shehane merged commit ed6f760 into cypress-io:master Mar 29, 2024
10 checks passed
@MikeMcC399 MikeMcC399 deleted the npm-branding branch March 30, 2024 07:21
@cypress-app-bot
Copy link
Collaborator

🎉 This PR is included in version 2.0.6 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants