Add typescript definitions

[jsamr]

Add index.d.ts :-)

[codecov]

Codecov Report

Merging #28 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master    #28   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files           1      1           
  Lines          28     28           
  Branches       11     11           
=====================================
  Hits           28     28

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e2c65f1...db71195. Read the comment docs.

[satazor]

Thanks for the effort in doing the PR but I will have to close it. While it does add the benefit of having typings for typescript users consuming this module, it will add a maintenance burden, specially since this module is not written in typescript.

Again, I’m sorry and hope you understand.

[jsamr]

@satazor ; well, I understand the maintenance burden you are consciously avoiding. However your API surface is very thin, and you could ping me on releases to get the job done.

Either that or I'll publish the definitions to definitely typed.

[satazor]

Exactly, it’s to avoid the maintenance burden. I know it’s thin but today is typescript, tomorrow is flow, and in the future is something else.

I would publish the definitions to definitively typed. 🙏

[jsamr]

@satazor DefinitelyTyped/DefinitelyTyped#39197
Should land soon.

[jsamr]

@satazor Please mention me on any API changes and I'll offer a PR in DT.

[jsamr]

https://www.npmjs.com/package/@types/proper-url-join