Skip to content
This repository has been archived by the owner on Aug 7, 2021. It is now read-only.

No Typescript typings #142

Closed
craxal opened this issue Dec 6, 2016 · 6 comments
Closed

No Typescript typings #142

craxal opened this issue Dec 6, 2016 · 6 comments

Comments

@craxal
Copy link

craxal commented Dec 6, 2016

Unless I'm mistaken, there doesn't seem to be any typings for node-adal. There is a @types/adal package on npm, but this appears to be for a different package. Having type declarations for this package would be greatly appreciated!

See also DefinitelyTyped/DefinitelyTyped#12586.

@balajikris
Copy link

balajikris commented Mar 30, 2017

Can someone from the team please comment on this? my team (azure-sdk-for-node) is prototyping a version of our libraries written purely in typescript and we depend on adal-node for authentication. Not having a .d.ts would be setback for us, so having some kind of idea of whether this will be supported or not will help us plan better. [tagging @amarzavery]

@mmaitre314
Copy link

I have a beginning of typings here: https://github.com/mmaitre314/CostMonitor/blob/master/CostMonitor/adal-node.d.ts
I can expand and send a PR if interested.

@SimonSchick
Copy link

Typings would be very helpful indeed.

@amarzavery
Copy link
Contributor

@rayluo @kpanwar - Can someone maintaining this repo provide an ETA for this?
FYI, There are projects like https://github.com/SitePen/dts-generator and https://github.com/Microsoft/dts-gen
that help you generate d.ts (type definition files).

@benrr101
Copy link

@amarzavery for the record, I just tried using the Microsoft/dts-gen to generate a .d.ts for adal-node -- no dice. I've logged the issue microsoft/dts-gen#84

@lovemaths
Copy link
Contributor

@craxal @balajikris @mmaitre314 @SimonSchick @benrr101 Typings added as of version 0.1.25. Many thanks to @amarzavery for the contribution.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

7 participants