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

Add typings to package.json. #10

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

Add typings to package.json. #10

wants to merge 1 commit into from

Conversation

ivasilov
Copy link

@ivasilov ivasilov commented May 6, 2016

According to this link the typings can be used directly from the NPM package, if specified in package.json. This PR adds support for it.

@spion
Copy link
Contributor

spion commented May 6, 2016

Are you sure they can be ambient external modules? Last time I checked you'd have to leave the wrapper out, e.g. if the original had

declare module "anydb-sql" {
// definitions here
}

you would need to remove the declare wrapper:

// definitions here

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.

2 participants