Skip to content

Commit d5d8c13

Browse files
ryan-williamsclaude
andcommitted
chore: drop npm-dist machinery — pkg.pr.new covers it
`pkg.pr.new` (via `cr.yml`) auto-publishes SHA-pinned previews on every push to main and works with any package manager, no `pnpm.overrides` gymnastics. The `dist` branch + `build-dist.yml` workflow are redundant. - Delete `.github/workflows/build-dist.yml` - Drop the "Alternate install: dist branch + pnpm.overrides" section from README - Trim the stale reference in `autofix.yml`'s comment Remote `dist` branch will be deleted in a follow-up `git push --delete`. `npm-dist` as its own project (https://github.com/runsascoded/npm-dist) still has independent value for single-package repos or non-hosted install paths. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent fa84470 commit d5d8c13

3 files changed

Lines changed: 5 additions & 56 deletions

File tree

.github/workflows/autofix.yml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -2,11 +2,11 @@ name: autofix.ci
22

33
# Disabled for the Open-Athena fork: autofix-ci/action fails with `The
44
# autofix.ci action is not allowed to modify the .github directory` on every
5-
# main push, because our fork edits workflow files (this one, `pages.yml`,
6-
# `build-dist.yml`) outside of autofix.ci's allowlist. We don't depend on the
7-
# bot to commit lint fixes from CI — the husky / lint-staged pre-commit hook
8-
# already runs eslint --fix locally. Manual `workflow_dispatch` stays
9-
# available if anyone wants to re-enable per-push autofix behaviour.
5+
# main push, because our fork edits workflow files (this one, `pages.yml`)
6+
# outside of autofix.ci's allowlist. We don't depend on the bot to commit lint
7+
# fixes from CI — the husky / lint-staged pre-commit hook already runs
8+
# `eslint --fix` locally. Manual `workflow_dispatch` stays available if anyone
9+
# wants to re-enable per-push autofix behaviour.
1010
on: workflow_dispatch
1111

1212
permissions:

.github/workflows/build-dist.yml

Lines changed: 0 additions & 31 deletions
This file was deleted.

README.md

Lines changed: 0 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -52,29 +52,9 @@ All three packages are required: `@slidev/cli` (the build/dev CLI) depends on `@
5252

5353
`@slidev/types` is unchanged in this fork — pull it straight from upstream npm if needed.
5454

55-
#### Alternate install: dist branch + `pnpm.overrides`
56-
57-
A second install path exists for `pnpm` users who'd rather pull from a git ref directly: the [`dist` branch] is auto-built per push and contains reified `package.json`s for each fork package.
58-
59-
```jsonc
60-
// package.json
61-
{
62-
"pnpm": {
63-
"overrides": {
64-
"@slidev/cli": "github:Open-Athena/slidev#dist&path:/packages/slidev",
65-
"@slidev/client": "github:Open-Athena/slidev#dist&path:/packages/client",
66-
"@slidev/parser": "github:Open-Athena/slidev#dist&path:/packages/parser"
67-
}
68-
}
69-
}
70-
```
71-
72-
Pin `#dist``#<sha>` for reproducible builds. Overrides are required (rather than direct deps) because the dist `@slidev/cli` references its siblings by exact upstream version — direct deps would semver-mismatch.
73-
7455
[`pkg.pr.new`]: https://pkg.pr.new
7556
[`cr.yml`]: https://github.com/Open-Athena/slidev/actions/workflows/cr.yml
7657
[`oa-slidev-theme`]: https://github.com/Open-Athena/oa-slidev-theme
77-
[`dist` branch]: https://github.com/Open-Athena/slidev/tree/dist
7858

7959
---
8060

0 commit comments

Comments
 (0)