Skip to content

Commit d3cfa5f

Browse files
ejntaylorclaude
andcommitted
Stop committing dist/ to the repo
The dist/ exception predates publishing to npm. Now that consumers install @patchstack/connect from the registry, the prepare script builds dist/ on install and the publish workflow builds it before packing — so there is no need to track build output in git. - Untrack dist/ and drop the .gitignore exception - Remove the "Verify committed dist is current" check from CI and publish workflows Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
1 parent bd497b2 commit d3cfa5f

11 files changed

Lines changed: 1 addition & 2950 deletions

File tree

.github/workflows/ci.yml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -49,9 +49,6 @@ jobs:
4949
- name: Build
5050
run: npm run build
5151

52-
- name: Verify committed dist is current
53-
run: git diff --exit-code -- dist
54-
5552
- name: Verify package contents
5653
run: npm pack --dry-run
5754

.github/workflows/publish.yml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -48,9 +48,6 @@ jobs:
4848
- name: Build
4949
run: npm run build
5050

51-
- name: Verify committed dist is current
52-
run: git diff --exit-code -- dist
53-
5451
- name: Verify package contents
5552
run: npm pack --dry-run
5653

.gitignore

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,9 +3,7 @@ node_modules
33
# used to exercise the node-modules walker. Keep those tracked.
44
!tests/fixtures/**/node_modules
55
!tests/fixtures/**/node_modules/**
6-
# dist/ is committed pre-npm-publish so consumers can `npm install github:patchstack/connect`
7-
# in sandboxed environments (WebContainer/StackBlitz/Bolt/Lovable) where the `prepare`
8-
# script can't run `tsup`. Remove this exception once the package is published to npm.
6+
dist
97
*.log
108
.DS_Store
119
.env

0 commit comments

Comments
 (0)