Skip to content

build: drop the git-install prepare hook - #60

Merged
TheGreatAxios merged 1 commit into
cl-9103-react-ui-ship-use-client-directives-on-stateful-componentsfrom
cl-9077-react-ui-drop-prepare-script-used-only-for-git-installs
Sep 27, 2026
Merged

TheGreatAxios merged 1 commit into
cl-9103-react-ui-ship-use-client-directives-on-stateful-componentsfrom
cl-9077-react-ui-drop-prepare-script-used-only-for-git-installs

Conversation

@TheGreatAxios

@TheGreatAxios TheGreatAxios commented Sep 25, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Stacked on #66, so it lands after #65 has removed the pin-a-commit install guidance.

  • scripts/prepare.mjs and the prepare script are gone. The package installs from npm, and prepack builds dist/ for the tarball.
  • IMPLEMENTATION.md no longer describes git installs. CHANGELOG lists "git installs are no longer supported" under breaking changes.

Verification

typecheck, lint, dep-guard, bun test and bun run build pass.

Closes CL-9077

@TheGreatAxios TheGreatAxios left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review · Approve

Removes the git-install prepare hook; prepack still builds dist/ for the npm tarball.

No findings.

@TheGreatAxios
TheGreatAxios force-pushed the cl-9077-react-ui-drop-prepare-script-used-only-for-git-installs branch from da4e0e0 to 31cddb0 Compare September 25, 2026 21:55
@TheGreatAxios
TheGreatAxios changed the base branch from main to cl-9103-react-ui-ship-use-client-directives-on-stateful-components September 25, 2026 21:55
@TheGreatAxios
TheGreatAxios force-pushed the cl-9077-react-ui-drop-prepare-script-used-only-for-git-installs branch from 31cddb0 to 173860e Compare September 26, 2026 01:32
@TheGreatAxios TheGreatAxios changed the title Drop the git-install prepare hook build: drop the git-install prepare hook Sep 26, 2026
@TheGreatAxios
TheGreatAxios force-pushed the cl-9077-react-ui-drop-prepare-script-used-only-for-git-installs branch from 173860e to 867578c Compare September 26, 2026 01:37
The package installs from npm; prepack builds dist for the tarball.
@TheGreatAxios
TheGreatAxios force-pushed the cl-9077-react-ui-drop-prepare-script-used-only-for-git-installs branch from 867578c to 5d0a253 Compare September 26, 2026 01:56
@TheGreatAxios
TheGreatAxios added this pull request to stack #74 September 26, 2026 01:57
@TheGreatAxios
TheGreatAxios merged commit 5cccb36 into main Sep 27, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant