Skip to content

Commit

Permalink
Maintenance, moving to cjs, and node20 (#57)
Browse files Browse the repository at this point in the history
* Updating dependencies.

* Moving to CJS, upping to node18

- Per convention, renaming all `.js` to `.cjs`
- Updating to NodeJS 18

* Delete jsconfig.json

Adds no value.

* Update action.yml

node18 isn't a choice, going to 20.

* Removing the `.js` file.

* Switching to esbuild

Swapping ncc out for esbuild
  • Loading branch information
manchicken committed Oct 1, 2023
1 parent 6e56083 commit d0cbd31
Show file tree
Hide file tree
Showing 34 changed files with 25,376 additions and 41,484 deletions.
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -3,3 +3,4 @@ tags
node_modules/*
coverage
.nova
**/.DS_Store
3 changes: 3 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,9 @@ and this project adheres to [Semantic Versioning].
- `@types/jest`
- `jest`
- `prettier`
- Using `cjs` extensions instead of `js` ones, per convention
- Updating to NodeJS 20
- Changing from `@vercel/ncc` to `esbuild` for packaging.

## [2.0.0] - 2023-06-29

Expand Down
66 changes: 0 additions & 66 deletions __tests__/TEST_PLANS.md

This file was deleted.

File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
4 changes: 2 additions & 2 deletions action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,8 @@ branding:
icon: 'monitor'
color: 'purple'
runs:
using: node16
main: dist/index.js
using: node20
main: dist/index.cjs
inputs:
# Stuff we need for this action
github-token:
Expand Down
Loading

0 comments on commit d0cbd31

Please sign in to comment.