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

fixes package name in docs #116

Closed
wants to merge 3 commits into from

Conversation

russbiggs
Copy link

This fixes the package name in the documentation. Currently the docs for the npm install lists the package as astro-Shield. Uppercase values are not allowed on npm, so this results in the following error:

npm ERR! code E404
npm ERR! 404 Not Found - GET https://registry.npmjs.org/@kindspells%2fastro-Shield - Not found
npm ERR! 404
npm ERR! 404  '@kindspells/astro-Shield@*' is not in this registry.
npm ERR! 404 This package name is not valid, because
npm ERR! 404  1. name can no longer contain capital letters
npm ERR! 404
npm ERR! 404 Note that you can also install from a
npm ERR! 404 tarball, folder, http url, or git url.

@castarco
Copy link
Contributor

Thank you @russbiggs , I'll be merging this today.

@castarco castarco closed this Sep 30, 2024
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