refc: rewrite the project in typescript

[gamemaker1]

Related Issues

• Closes Add ESLint and move to TypeScript #705

Description

This PR:

• rewrites the library in Typescript
• makes this package an ES module.
• adds ESLint and Prettier
• adds Husky and Lint Staged
• adds a contributing guide
• adds lint and test steps to the GitHub workflow
• adds issue and pull request templates

Checklist

• The issues that this PR fixes/closes have been mentioned above.
• What this PR adds/changes/removes has been explained.
• All tests (pnpm test) pass.
• The linter (pnpm lint) does not throw an errors.
• All added/modified code has been commented, and
  methods/classes/constants/types have been annotated with TSDoc comments.

[gamemaker1]

I've tried to stick as close to the style guide as possible, but please let me know if there's something I missed and should change!

[leerob]

This is awesome! Going to dig into this and give it a thorough review.

[schemburkar]

@gamemaker1 I have ported my work in #580 to code based on your current PR. How do you want me to push this? Directly to this PR or separate one?
cc: @leerob

[gamemaker1]

@schemburkar That was really fast 🎉 Thanks for the amazing work on the PR!

I think it makes sense to keep the certificate PR separate and merge it in after this PR.

I want to make a couple of other PRs too, like the logging one and changes from useful forks like https://github.com/warren-bank/node-serve, but I'll wait until this one is merged and then make those separately. It'll make it easier for the Vercel team to review changes :]

[leerob]

Everything is working as expected 👍

[mrmckeb]

This looks great, a huge effort - thanks!

Let me know if you want help or want to pair on any of the comments, more than happy to help with any of it.

[gamemaker1]

I have to step away for now, but once I am bacj I will address the ts-expect-error and eslint-disable comments as @mrmckeb pointed out. Thanks for the explanation and the hints @mrmckeb!

[gamemaker1]

Removed all the ts-expect-error and eslint-disable comments that @mrmckeb pointed out. Thanks for pointing it out and helping me remove them!

[gamemaker1]

Any other changes I should make?

[leerob]

All of my comments have been addressed. To reiterate, we will be releasing this as a major version and can write a nice summary in the release notes to let folks know what changed.

Let's get two other +1s and we can ship it.

[gamemaker1]

That's great!

Once this PR is merged, I would love to help out with adding the tests and some features from forks like https://github.com/warren-bank/node-serve; looking forward to collaborating with all of you on it :]

[mrmckeb]

This is great. I'm not an expert on this package, but the changes you've made look logical and you've done a great job of typing and adhering to our style guide - thank you!

[leerob]

Thank you @gamemaker1 🙏 Would love to send you some Vercel swag (shoot me an email [email protected]).

[leerob]

[email protected] published to npm.

[leerob]

Confirmed things are working as expected, publishing stable – will keep an eye on issues 😄

[leerob]

So far I've only seen one small bug: #711

[gamemaker1]

Would love to send you some Vercel swag (shoot me an email).

Wow, thank you! I will send you an email today :]