Skip to content

Commit c7127e5

Browse files
committed
ci: improve GitHub Actions and docs
- Update Node.js versions to match requirements - Add explicit Node-RED installation - Add test environment variable - Split steps for better error tracking - Fix NIP-40 formatting in README
1 parent 63c3ba7 commit c7127e5

File tree

2 files changed

+17
-5
lines changed

2 files changed

+17
-5
lines changed

.github/workflows/test.yml

Lines changed: 16 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -12,15 +12,27 @@ jobs:
1212

1313
strategy:
1414
matrix:
15-
node-version: [14.x, 16.x, 18.x]
15+
node-version: [16.x, 18.x, 20.x]
1616

1717
steps:
1818
- uses: actions/checkout@v3
19+
1920
- name: Use Node.js ${{ matrix.node-version }}
2021
uses: actions/setup-node@v3
2122
with:
2223
node-version: ${{ matrix.node-version }}
2324
cache: 'npm'
24-
- run: npm ci
25-
- run: npm run build --if-present
26-
- run: npm test
25+
26+
- name: Install dependencies
27+
run: npm ci
28+
29+
- name: Install Node-RED (required for tests)
30+
run: npm install -g node-red@latest
31+
32+
- name: Build
33+
run: npm run build
34+
35+
- name: Run tests
36+
run: npm test
37+
env:
38+
NODE_ENV: test

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -187,7 +187,7 @@ docker compose -f docker-compose.dev.yml up
187187
| [NIP-15](https://github.com/nostr-protocol/nips/blob/master/15.md) | End of Stored Events Notice | ✅ Implemented |
188188
| [NIP-20](https://github.com/nostr-protocol/nips/blob/master/20.md) | Command Results | ✅ Implemented |
189189
| [NIP-28](https://github.com/nostr-protocol/nips/blob/master/28.md) | Public Chat | 🚧 Planned |
190-
| [40](https://github.com/nostr-protocol/nips/blob/master/40.md) | Expiration Timestamp | 🚧 Planned |
190+
| [NIP-40](https://github.com/nostr-protocol/nips/blob/master/40.md) | Expiration Timestamp | 🚧 Planned |
191191

192192
Legend:
193193
- ✅ Implemented: Fully supported

0 commit comments

Comments
 (0)