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

Fix bunchee command execution in readme #466

Merged
merged 2 commits into from
Mar 6, 2024
Merged

Conversation

piotr-cz
Copy link
Contributor

Executing with npm results in

npm bunchee --prepare

Unknown command: "bunchee"

To see a list of supported npm commands, run:
  npm help

@huozhi
Copy link
Owner

huozhi commented Mar 3, 2024

That one is by purpose for the installed case. there's another line below with npx case

# "Or use with npx"
# npx bunchee@latest --prepare

@huozhi huozhi closed this Mar 3, 2024
@piotr-cz
Copy link
Contributor Author

piotr-cz commented Mar 5, 2024

@huozhi

Command fails when bunchee is installed.

To reproduce, try:

npm install bunchee
npm bunchee --prepare

@huozhi
Copy link
Owner

huozhi commented Mar 5, 2024

oh that works with pnpm, for npm I guess we need to do npm exec bunchee --prepare?

@piotr-cz
Copy link
Contributor Author

piotr-cz commented Mar 5, 2024

@huozhi
Yes, for npm works both npm exec bunchee --prepare and npx bunchee --prepare

@huozhi huozhi reopened this Mar 6, 2024
README.md Outdated Show resolved Hide resolved
@huozhi huozhi merged commit ba53d9a into huozhi:main Mar 6, 2024
2 checks passed
@piotr-cz piotr-cz deleted the patch-1 branch March 8, 2024 07:56
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