Skip to content

fix(vite-plugin): exit jahia-deploy with code 1 on failure - #813

Merged
GauBen merged 1 commit into
mainfrom
fix/jahia-deploy-exit-1
Oct 14, 2026
Merged

GauBen merged 1 commit into
mainfrom
fix/jahia-deploy-exit-1

Conversation

@GauBen

@GauBen GauBen commented Oct 14, 2026

Copy link
Copy Markdown
Member

Description

Exit jahia-deploy with code 1 on failure

Needed until the provisioning API does it for us

Checklist

Source code

  • I've shared and documented any breaking change
  • I've reviewed and updated the jahia-depends

Tests

  • I've provided Unit and/or Integration Tests
  • I've updated the parent issue with required manual validations

Tip

Documentation to guide the reviews: How to do a code review

@GauBen
GauBen requested review from jkevan and a balanced review from Copilot October 14, 2026 07:41
@github-actions

Copy link
Copy Markdown

🦜 Chachalog

javascript-modules patch
  • jahia-deploy now fails when Jahia installs nothing, instead of printing {} and exiting successfully.

Create a new entry online or run npx chachalog@0.5.6 prompt to create a new entry locally.

@GauBen
GauBen removed the request for review from Copilot October 14, 2026 07:42
@pkg-pr-new

pkg-pr-new Bot commented Oct 14, 2026

Copy link
Copy Markdown

Open in StackBlitz

yarn add https://pkg.pr.new/@jahia/create-module@813.tgz
yarn add https://pkg.pr.new/@jahia/javascript-modules-library@813.tgz
yarn add https://pkg.pr.new/@jahia/vite-plugin@813.tgz

commit: 2e491a2

@GauBen
GauBen merged commit beada7c into main Oct 14, 2026
43 checks passed
@GauBen
GauBen deleted the fix/jahia-deploy-exit-1 branch October 14, 2026 08:25
@Fgerthoffert Fgerthoffert mentioned this pull request Oct 14, 2026
13 of 24 tasks
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