Skip to content

docs: add enablePrePostScripts option to run config page#801

Merged
branchseer merged 3 commits intomainfrom
claude/fix-ci-pr-733-8Cebv
Mar 12, 2026
Merged

docs: add enablePrePostScripts option to run config page#801
branchseer merged 3 commits intomainfrom
claude/fix-ci-pr-733-8Cebv

Conversation

@branchseer
Copy link
Member

@branchseer branchseer commented Mar 12, 2026

Summary

Test plan

  • Docs render correctly
  • CI passes

Documents the new `run.enablePrePostScripts` option that controls
automatic pre/post lifecycle hooks for package.json scripts.

https://claude.ai/code/session_0139LsSbe67hcD8NKKaK6Y8U
@netlify
Copy link

netlify bot commented Mar 12, 2026

Deploy Preview for viteplus-staging ready!

Name Link
🔨 Latest commit 14101c9
🔍 Latest deploy log https://app.netlify.com/projects/viteplus-staging/deploys/69b2bdc50664be0008545803
😎 Deploy Preview https://deploy-preview-801--viteplus-staging.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@branchseer branchseer requested a review from cpojer March 12, 2026 11:01
@cpojer
Copy link
Member

cpojer commented Mar 12, 2026

Can you walk me through why we are adding this option? It is on by default, and I don't see why anyone would want to disable it?

Why not call it "disableHookScripts" instead (default: false).

Not going to block this, I'm sure there are reasons, but it seems kinda too much to add this option that we'll have to support for 10 years.

@branchseer branchseer merged commit b15e59e into main Mar 12, 2026
17 checks passed
@branchseer branchseer deleted the claude/fix-ci-pr-733-8Cebv branch March 12, 2026 13:21
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.

4 participants