Skip to content

Support command config in either "commands" or "command"#597

Merged
hzoo merged 1 commit intolerna:masterfrom
gigabo:command-config
Feb 13, 2017
Merged

Support command config in either "commands" or "command"#597
hzoo merged 1 commit intolerna:masterfrom
gigabo:command-config

Conversation

@gigabo
Copy link
Copy Markdown
Contributor

@gigabo gigabo commented Feb 10, 2017

Noticed that the docs in the README say it's under "commands", but the code
was only looking in "command". This patch supports either/or.

Example:

{
  ...
  "commands": {
    "bootstrap": {
      "includeFilteredDependencies": true
    }
  }
}

Noticed that the docs in the README say it's under "commands", but the code
was only looking in "command".  This patch supports either/or.

Example:

```js
{
  ...
  "commands": {
    "bootstrap": {
      "includeFilteredDependencies": true
    }
  }
}
@hzoo
Copy link
Copy Markdown
Contributor

hzoo commented Feb 13, 2017

We can make the breaking change but ok +1

@hzoo hzoo merged commit 2acb721 into lerna:master Feb 13, 2017
@lock
Copy link
Copy Markdown

lock Bot commented Dec 27, 2018

This thread has been automatically locked because there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock Bot locked as resolved and limited conversation to collaborators Dec 27, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants