Skip to content
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 0 additions & 5 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -39,11 +39,6 @@ jobs:

- run: bun install --frozen-lockfile

# No unpublished @workbench/* scope may leak in — it does not exist on
# npm, so a leak makes the package uninstallable outside the monorepo.
- name: check-deps
run: bun run scripts/check-deps.ts

# typecheck builds first: examples import @corbits/artifacts through the
# published dist types, same path a consumer resolves.
- name: typecheck
Expand Down
2 changes: 2 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -89,6 +89,8 @@ always called out under their own heading.

### Changed

- The `scripts/check-deps.ts` pretest is removed; typecheck and tests already
fail on an unresolvable `@workbench/*` import.
- Minimum `@intx/*` is now **0.3.0**. (`@intx/*` lines before 0.3.0 do not
install — older lines pin the unpublished `@intx/*@0.0.0` or ship raw
TypeScript.)
Expand Down
3 changes: 1 addition & 2 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -35,8 +35,7 @@ reference host is where that change has to be shown working.
## Dependency rule

No `@workbench/*` imports anywhere — it is an unpublished scope, and importing it would
make this package uninstallable outside the project that defines it. Checked by
`scripts/check-deps.ts`, which runs as `pretest` and again in CI.
make this package uninstallable outside the project that defines it.

## Tests

Expand Down
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -141,7 +141,7 @@ docker run -d --name corbits-artifact-pg -p 5457:5432 \
export ALLOW_DESTRUCTIVE_ARTIFACT_TESTS=1

bun run typecheck
bun run test # pretest dependency check, then unit + integration
bun run test # unit + integration
bun run build # dist/ (JS + .d.ts)
bun run test:acceptance # builds, then examples/reference-host
```
Expand Down
5 changes: 2 additions & 3 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -55,9 +55,8 @@
"typecheck": "tsc --noEmit",
"build": "rm -rf dist && tsc -p tsconfig.build.json",
"prepack": "bun run build",
"pretest": "bun run scripts/check-deps.ts",
"test": "bun run pretest && bun test src",
"test:coverage": "bun run pretest && bun test --coverage src",
"test": "bun test src",
"test:coverage": "bun test --coverage src",
"test:package": "bun run test",
"test:acceptance": "bun run build && bun test --cwd examples/reference-host"
},
Expand Down
19 changes: 0 additions & 19 deletions scripts/check-deps.ts

This file was deleted.

2 changes: 1 addition & 1 deletion tsconfig.json
Original file line number Diff line number Diff line change
Expand Up @@ -10,5 +10,5 @@
"noEmit": true,
"types": ["node", "@types/bun"]
},
"include": ["src", "scripts"]
"include": ["src"]
}
Loading