Thanks for looking to contribute,@aboyton! I'm afraid I don't understand the goal of this PR, and it is fundamentally broken. You are adding imports and exports to node .js files, that will not work. Also"resolveJsonModule": true will alter the structure of the build output, so has significant repercussions.

These are not changes we can make right now but thanks again for supporting the project

All good. I was looking for an easy thing to get started on the project again and noticed that#67 had a comment

i had to disable some of recommended rules, and we should fix issues reported by them.

and so I figured I'd try see if I could turn some of them on. I wasn't sure if this would break anything or not, I ran the tests and then figured why not see if this was correct or useful. Sorry to waste your time.

Oh not at all, I really appreciate the offer of help! I'm afraid the new setup is just a little too new for us to have figured out good contributing guidance etc.

We'll definitely improve on that and hopefully we'll see some more PRs from you in the near future :)

idea of turning off@typescript-eslint/no-var-requires in js only files seems valid

@aboyton you can try updating it in ts files only and turning this rule off for all js files