Skip to content

Commit

Permalink
Migrates to modules for reduced build size
Browse files Browse the repository at this point in the history
* Migrates require to import
* Switches from vercel/ncc to esbuild to account for dependency require
* Upgrades deps to latest
* Adds test:watch
* Adds test to make sure build isn't out of date
  • Loading branch information
joncloud committed Jan 22, 2023
1 parent b235abd commit 9e6ef6e
Show file tree
Hide file tree
Showing 21 changed files with 809 additions and 3,459 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/node.js.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,10 +17,10 @@ jobs:
node-version: [16.x]

steps:
- uses: actions/checkout@v2
- uses: actions/checkout@v3

- name: Use Node.js ${{ matrix.node-version }}
uses: actions/setup-node@v1
uses: actions/setup-node@v3
with:
node-version: ${{ matrix.node-version }}

Expand Down Expand Up @@ -54,6 +54,6 @@ jobs:
# If this fails, don't worry the JSON output will be printed as a backup.
continue-on-error: true
with:
name: 'Tests ${{ matrix.os }}'
name: 'test (${{ matrix.os }}, ${{ matrix.node-version }})'
path: ${{ runner.temp }}/report-${{ matrix.os }}.json
reporter: mocha-json
5 changes: 5 additions & 0 deletions .mocharc.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
{
"node-option": [
"experimental-specifier-resolution=node"
]
}
2 changes: 1 addition & 1 deletion .nvmrc
Original file line number Diff line number Diff line change
@@ -1 +1 @@
v16.15.1
16
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ Newline-delimited list of paths to load plugins from. Default `""`.

```yml
- name: Create installer
uses: joncloud/makensis-action@v3.7
uses: joncloud/makensis-action@v4
with:
arguments: "/V3"
```
Expand Down
2 changes: 1 addition & 1 deletion action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ inputs:

runs:
using: 'node16'
main: 'dist/index.js'
main: 'dist/index.cjs'

branding:
icon: 'triangle'
Expand Down
8 changes: 8 additions & 0 deletions dist/index.cjs

Large diffs are not rendered by default.

Loading

0 comments on commit 9e6ef6e

Please sign in to comment.