-
Notifications
You must be signed in to change notification settings - Fork 1k
Include --ignore-optional doc #271
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
base: master
Are you sure you want to change the base?
Conversation
Resolves yarnpkg#265. Two changes: 1. Fix broken link to optionalDependencies 2. Include documentation for the --ignore-optional CLI command
Shouldn't this be on the |
@thejameskyle This is in reference to
However I see the same list when I run
|
Internally add uses a lot of the same logic that install has, I wouldn't be surprised if was using the same content even if some of those flags don't make sense in @kittens did you intend this? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be in the documentation for yarn install
, not yarn add
.
@Daniel15 I moved this to the |
The PR looks good to me, but given yarnpkg/yarn#2666 it could be good clarify whether |
Resolves #265.
Two changes:
Looks like this (locally):