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: yarn warning for invalid bin entry #65

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

eps1lon
Copy link

@eps1lon eps1lon commented Jan 19, 2020

BREAKING CHANGE:
Removed the dot in the binary name.

- $ sha.js
+ $ shajs

Closes #64

Can be verified locally. A yarn in the source directory will not warn about invalid bin entries on this branch.

@@ -18,7 +18,9 @@
"tape": "~2.3.2",
"typedarray": "0.0.6"
},
"bin": "./bin.js",
"bin": {
"shajs": "./bin.js"
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You could wait for yarnpkg/yarn#7811 to be released. This might allow to fix this without a breaking change.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks man, this was already merged. I hope the author will finally merge this one here ?

@earonesty
Copy link

note: no need to merge this, since it's a yarn issue... and has been fixed upstream

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.

New Yarn version, package.json bin causes warning
3 participants