Skip to content

Commit a4af08d

Browse files
authored
Fix NCList having string phase field sometimes (#41)
1 parent cfc09a0 commit a4af08d

15 files changed

Lines changed: 6058 additions & 4865 deletions

.github/workflows/publish.yml

Lines changed: 27 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,27 @@
1+
name: Publish
2+
3+
on:
4+
push:
5+
tags:
6+
- 'v*'
7+
8+
jobs:
9+
publish:
10+
name: Publish to npm
11+
runs-on: ubuntu-latest
12+
permissions:
13+
id-token: write
14+
contents: read
15+
steps:
16+
- uses: actions/checkout@v4
17+
- uses: pnpm/action-setup@v4
18+
with:
19+
version: 10
20+
- uses: actions/setup-node@v4
21+
with:
22+
node-version: 24.x
23+
cache: pnpm
24+
registry-url: 'https://registry.npmjs.org'
25+
- run: pnpm install --frozen-lockfile
26+
- run: pnpm build
27+
- run: npm publish --access public

.github/workflows/push.yml

Lines changed: 11 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -4,21 +4,18 @@ on: push
44

55
jobs:
66
test:
7-
name: Lint, build, and test on node 20.x and ubuntu-latest
7+
name: Lint, build, and test
88
runs-on: ubuntu-latest
99
steps:
1010
- uses: actions/checkout@v4
11-
- name: Use Node.js 20.x
12-
uses: actions/setup-node@v4
11+
- uses: pnpm/action-setup@v4
1312
with:
14-
node-version: 20.x
15-
- name: Install deps (with cache)
16-
uses: bahmutov/npm-install@v1
17-
- name: Lint codebase
18-
run: yarn lint
19-
- name: Build codebase
20-
run: yarn build
21-
- name: Test codebase
22-
run: yarn test --coverage
23-
- name: Upload coverage
24-
run: bash <(curl -s https://codecov.io/bash)
13+
version: 10
14+
- uses: actions/setup-node@v4
15+
with:
16+
node-version: 22.x
17+
cache: pnpm
18+
- run: pnpm install --frozen-lockfile
19+
- run: pnpm lint
20+
- run: pnpm build
21+
- run: pnpm test

.prettierignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1 +1,2 @@
11
test/data
2+
pnpm-lock.yaml

eslint.config.mjs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ export default defineConfig(
1212
'esm/*',
1313
'example/*',
1414
'eslint.config.mjs',
15+
'vitest.config.ts',
1516
],
1617
},
1718
{

package.json

Lines changed: 26 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,10 @@
1212
"require": "./dist/index.js"
1313
}
1414
},
15-
"repository": "github:GMOD/nclist-js",
15+
"repository": {
16+
"type": "git",
17+
"url": "https://github.com/GMOD/nclist-js.git"
18+
},
1619
"author": {
1720
"name": "Robert Buels",
1821
"email": "rbuels@gmail.com",
@@ -28,42 +31,41 @@
2831
"esm",
2932
"src"
3033
],
31-
"engines": {
32-
"node": ">=8"
33-
},
3434
"scripts": {
35+
"format": "prettier --write .",
3536
"test": "vitest",
36-
"coverage": "yarn test --coverage",
37+
"coverage": "pnpm test --coverage",
3738
"lint": "eslint --report-unused-disable-directives --max-warnings 0",
3839
"docs": "documentation readme --shallow src/feature_store.ts --section API",
3940
"clean": "rimraf dist esm",
40-
"prebuild": "yarn docs && yarn clean && yarn lint",
41+
"prebuild": "pnpm docs && pnpm clean && pnpm lint",
4142
"build:esm": "tsc --outDir esm",
42-
"build:es5": "tsc --module commonjs --outDir dist",
43+
"build:es5": "tsc --module commonjs --moduleResolution bundler --outDir dist",
4344
"postbuild:es5": "echo '{\"type\": \"commonjs\"}' > dist/package.json",
44-
"build": "yarn build:esm && yarn build:es5",
45-
"prepublishOnly": "yarn test --run && yarn build",
45+
"build": "pnpm build:esm && pnpm build:es5",
46+
"prepublishOnly": "pnpm test --run && pnpm build",
4647
"postversion": "git push --follow-tags"
4748
},
4849
"devDependencies": {
49-
"@types/express": "^5.0.5",
50-
"@vitest/coverage-v8": "^4.0.6",
50+
"@eslint/js": "^10.0.1",
51+
"@types/express": "^5.0.6",
52+
"@vitest/coverage-v8": "^4.1.2",
5153
"cross-fetch": "^4.1.0",
52-
"documentation": "^14.0.1",
53-
"eslint": "^9.5.0",
54-
"eslint-plugin-import": "^2.31.0",
55-
"eslint-plugin-unicorn": "^62.0.0",
56-
"express": "^5.1.0",
57-
"generic-filehandle2": "^2.0.1",
58-
"get-port": "^7.1.0",
59-
"prettier": "^3.5.3",
60-
"rimraf": "^6.0.1",
61-
"typescript": "^5.0.4",
62-
"typescript-eslint": "^8.18.0",
63-
"vitest": "^4.0.6"
54+
"documentation": "^14.0.3",
55+
"eslint": "^9.39.4",
56+
"eslint-plugin-import": "^2.32.0",
57+
"eslint-plugin-unicorn": "^64.0.0",
58+
"express": "^5.2.1",
59+
"generic-filehandle2": "^2.1.4",
60+
"get-port": "^7.2.0",
61+
"prettier": "^3.8.1",
62+
"rimraf": "^6.1.3",
63+
"typescript": "^6.0.2",
64+
"typescript-eslint": "^8.58.0",
65+
"vitest": "^4.1.2"
6466
},
6567
"dependencies": {
66-
"@gmod/abortable-promise-cache": "^3.0.1",
68+
"@gmod/abortable-promise-cache": "^3.0.4",
6769
"@jbrowse/quick-lru": "^7.3.5",
6870
"@jridgewell/resolve-uri": "^3.1.2"
6971
},

0 commit comments

Comments
 (0)