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

TypeScript and ESM support #276

Open
8 tasks
coopernetes opened this issue Sep 4, 2023 · 4 comments
Open
8 tasks

TypeScript and ESM support #276

coopernetes opened this issue Sep 4, 2023 · 4 comments
Labels
help wanted Extra attention is needed

Comments

@coopernetes
Copy link
Contributor

coopernetes commented Sep 4, 2023

Tracking issue to convert the project to TypeScript from JS, convert existing CommonJS-based source code to ES modules and update tooling. This will include a number of tasks including:

  • adding TS compilation
  • changing existing CJS structure to ESM
    • convert require into import
    • convert module.exports with export default statements
  • update linting to add TS support
    • workflows
    • hook scripts
    • configurations

@coopernetes - Yes, we should spin up an issue for this 👍 It will be better practice for our development of the library, as well as for anyone else looking to contribute or build plugins.

Can we spin up an issue to introduce TypeScript to the project with linting support?

Originally posted by @JamieSlome in #47 (comment)

@JamieSlome JamieSlome added help wanted Extra attention is needed good first issue Good for newcomers labels Sep 5, 2023
@JamieSlome
Copy link
Member

@coopernetes - thanks for creating the issue! 👍

@JamieSlome JamieSlome changed the title feat: TypeScript and ESM support TypeScript and ESM support Oct 31, 2023
@coopernetes coopernetes removed the good first issue Good for newcomers label Jan 14, 2024
@LewisBroadhurst
Copy link

How's this task looking? I'm up for getting started on it

@coopernetes
Copy link
Contributor Author

Hi @LewisBroadhurst I will be creating a new branch in this repo which PRs will be accepted into. I have some prior art to share on this issue so let me get that pushed out in the next day or two so we can start the work to close out this issue.

@coopernetes
Copy link
Contributor Author

New branch is up with a preliminary tsconfig.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants