-
Notifications
You must be signed in to change notification settings - Fork 1
Open
Description
Oh node, how we love thee.
In my work on #11 I'm moving the project to type: "module" which lets us do things like export foo() in a utility file.
Since it never hurts to move into modern node, I'd like to propose we consider:
- Adding babel
- Adding eslint (we have eslint in the project, though it is not clear when or how to run it
- Adding tests (I'm most familiar with jest)
Once we do this we can consider adding some CI to invoke the linter + tests before allowing a PR to merge. This will all be especially useful as we consider #18
Metadata
Metadata
Assignees
Labels
No labels