Skip to content

Conversation

@Kristinita
Copy link
Contributor

@Kristinita Kristinita commented Apr 10, 2018

1. Summary

Fix installation syntax for Windows.

Curly braces syntax doesn't work for Windows-specific terminals.

2. Behavior before pull request

D:\SashaDemoRepositories\SashaCommitlint>npm install --save-dev @commitlint/{config-conventional,cli}
npm ERR! code ENOLOCAL
npm ERR! Could not install from "@commitlint\{config-conventional,cli}" as it does not contain a package.json file.

npm ERR! A complete log of this run can be found in:
npm ERR!     C:\Users\SashaChernykh\AppData\Roaming\npm-cache\_logs\2018-04-10T13_23_19_766Z-debug.log

3. Behavior after pull request

D:\SashaDemoRepositories\SashaCommitlint>npm install --save-dev @commitlint/config-conventional @commitlint/cli

+ @commitlint/config-conventional@6.1.3
+ @commitlint/cli@6.1.3
added 170 packages from 61 contributors in 34.977s

4. Testing environment

  • Windows 10 Enterprise LTSB 64-bit EN,
  • Node.js 9.11.1,
  • npm 5.8.0,
  • Clink 0.4.8.

Thanks.

### 1. Summary

Fix installation syntax for Windows.

Curly braces syntax [**doesn't work**](https://stackoverflow.com/a/38876970/5951529) for Windows-specific terminals.

### 2. Behavior before pull request

```node
D:\SashaDemoRepositories\SashaCommitlint>npm install --save-dev @commitlint/{config-conventional,cli}
npm ERR! code ENOLOCAL
npm ERR! Could not install from "@commitlint\{config-conventional,cli}" as it does not contain a package.json file.

npm ERR! A complete log of this run can be found in:
npm ERR!     C:\Users\SashaChernykh\AppData\Roaming\npm-cache\_logs\2018-04-10T13_23_19_766Z-debug.log
```

### 3. Behavior after pull request

```node
D:\SashaDemoRepositories\SashaCommitlint>npm install --save-dev @commitlint/config-conventional @commitlint/cli

+ @commitlint/[email protected]
+ @commitlint/[email protected]
added 170 packages from 61 contributors in 34.977s
```

### 4. Testing environment

+ Windows 10 Enterprise LTSB 64-bit EN,
+ Node.js 9.11.1,
+ npm 5.8.0,
+ [**Clink**](https://mridgers.github.io/clink/) 0.4.8.

Thanks.
@marionebl
Copy link
Contributor

Thanks!

@marionebl marionebl merged commit b0bbce3 into conventional-changelog:master Apr 10, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants