Skip to content

Conversation

@pwbriggs
Copy link
Contributor

@pwbriggs pwbriggs commented Feb 26, 2024

What's the problem this PR addresses?

In the website search placeholder text (something like Search packages (i.e. babel, webpack, react, ...)), the word "i.e." is used incorrectly. "i.e." means "that is," but the intended meaning here is "e.g.," meaning "for example."

How did you fix it?

Changed the three instances I found of this placeholder text to use "e.g." instead.

Checklist

  • I have set the packages that need to be released for my changes to be effective.
    👆 Not sure how to do that / if it's applicable to docs-only changes. never mind, see comment below.
  • I will check that all automated PR checks pass before the PR gets reviewed.

In the website search placeholder text (something like "Search packages (i.e. babel, webpack, react, ...)"), i.e. is used incorrectly. "I.e." means "that is," whereas in this case it should be "e.g.", meaning "for example."
This patch was only intended to change the website.
@pwbriggs
Copy link
Contributor Author

Sorry, I just realized that I changed a search placeholder that wasn't on the website. I undid that change, so now the tests should pass. (We should also update that placeholder in plugin-interactive-tools, but I think that belongs in another PR.)

@arcanis arcanis merged commit b880acf into yarnpkg:master Mar 4, 2024
@pwbriggs pwbriggs deleted the ie-to-eg branch March 4, 2024 21:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants