Skip to content
This repository has been archived by the owner on Nov 16, 2023. It is now read-only.

Allow dtslint types @bla/foo #47

Merged
1 commit merged into from
Aug 14, 2017
Merged

Allow dtslint types @bla/foo #47

1 commit merged into from
Aug 14, 2017

Conversation

joscha
Copy link
Contributor

@joscha joscha commented Jun 5, 2017

Currently when running npm run lint @bla/foo in https://github.com/DefinitelyTyped/DefinitelyTyped it errors with:

➜  DefinitelyTyped git:(add-@storybook__react) ✗ npm run lint @storybook/react

> [email protected] lint /Users/joscha/Development/DefinitelyTyped
> dtslint types "@storybook/react"

ENOENT: no such file or directory, open '/Users/joscha/Development/DefinitelyTyped/types/@storybook/react/index.d.ts'

npm ERR! Darwin 16.6.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "run" "lint" "@storybook/react"
npm ERR! node v7.5.0
npm ERR! npm  v4.2.0
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR! [email protected] lint: `dtslint types "@storybook/react"`
npm ERR! Exit status 1

for scoped packages.

references microsoft/dts-gen#63, DefinitelyTyped/dt-review-tool#11

Workaround is currently to run npm run lint storybook__react

Currently when running `npm run lint @bla/foo` in https://github.com/DefinitelyTyped/DefinitelyTyped it errors with:

```
➜  DefinitelyTyped git:(add-@storybook__react) ✗ npm run lint @storybook/react

> [email protected] lint /Users/joscha/Development/DefinitelyTyped
> dtslint types "@storybook/react"

ENOENT: no such file or directory, open '/Users/joscha/Development/DefinitelyTyped/types/@storybook/react/index.d.ts'

npm ERR! Darwin 16.6.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "run" "lint" "@storybook/react"
npm ERR! node v7.5.0
npm ERR! npm  v4.2.0
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR! [email protected] lint: `dtslint types "@storybook/react"`
npm ERR! Exit status 1
```

for scoped packages.

references microsoft/dts-gen#63
@msftclas
Copy link

msftclas commented Jun 5, 2017

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@joscha
Copy link
Contributor Author

joscha commented Jun 5, 2017

references DefinitelyTyped/DefinitelyTyped#16970

@ghost
Copy link

ghost commented Aug 14, 2017

Thanks!

@ghost ghost merged commit 0a43b6f into microsoft:master Aug 14, 2017
@joscha joscha deleted the patch-1 branch August 14, 2017 23:26
This pull request was closed.
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