Skip to content

Modify pre-build hook to include npm install#970

Open
clemcode wants to merge 1 commit into
CleverCloud:mainfrom
clemcode:patch-1
Open

Modify pre-build hook to include npm install#970
clemcode wants to merge 1 commit into
CleverCloud:mainfrom
clemcode:patch-1

Conversation

@clemcode

Copy link
Copy Markdown

📝 What does this PR do?

Modify pre-build hook to include npm install after following the docs and getting a command not found error on build

🧪 Type of Change

  • ⚠️ Bug fix
  • 📅 Changelog update
  • 📚 Documentation update
  • ✨ New content/feature
  • 🔧 Technical/maintenance

✅ Quick Checklist


👥 Reviewers

@CleverCloud/reviewers

@github-actions

github-actions Bot commented Jun 16, 2026

Copy link
Copy Markdown

✅ Review app deployed

Check how your changes look like!

ℹ️ Name 🔗 Infos & links
🌱 Latest deployed commit 56a5abd
👁️ Preview https://documentation-PR-970.cleverapps.io/developers

@davlgd

davlgd commented Jun 17, 2026

Copy link
Copy Markdown
Collaborator

There is no need to add npm install, as it's automatically done during the build phase on a Node.js & Bun application. I suppose the problem here is more that the npm run build command is launched during the pre-build hook (so before the npm install).

Can you try to replace this command with clever env set CC_POST_BUILD_HOOK "npm run build" and tell me if it works ? (or I'll try later to check).

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.

2 participants