Skip to content

feat: add npm.excludePostinstallPackages option#24421

Merged
caalador merged 2 commits into
mainfrom
fix/2433-add-npm-excludePostinstallPackages
May 26, 2026
Merged

feat: add npm.excludePostinstallPackages option#24421
caalador merged 2 commits into
mainfrom
fix/2433-add-npm-excludePostinstallPackages

Conversation

@tltv
Copy link
Copy Markdown
Member

@tltv tltv commented May 22, 2026

Adds a configuration option to exclude packages from running postinstall scripts. Mirrors the existing npm.postinstallPackages add-list and applies to both the built-in default entries (e.g. esbuild, @vaadin/vaadin-usage-statistics) and any packages added via postinstallPackages. Wired through Options, the Maven and Gradle plugins, and DevModeInitializer.

Refs #24333

Adds a configuration option to exclude packages from running postinstall
scripts. Mirrors the existing npm.postinstallPackages add-list and applies
to both the built-in default entries (e.g. esbuild,
@vaadin/vaadin-usage-statistics) and any packages added via
postinstallPackages. Wired through Options, the Maven and Gradle plugins,
and DevModeInitializer.

Refs #24333
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 22, 2026

Test Results

 1 418 files  ±0   1 418 suites  ±0   1h 22m 3s ⏱️ +47s
10 000 tests ±0   9 931 ✅ ±0  69 💤 ±0  0 ❌ ±0 
10 475 runs  ±0  10 404 ✅ ±0  71 💤 ±0  0 ❌ ±0 

Results for commit f3c3581. ± Comparison against base commit eef1d26.

♻️ This comment has been updated with latest results.

@sonarqubecloud
Copy link
Copy Markdown

@mshabarov mshabarov requested a review from caalador May 25, 2026 11:45
@caalador caalador added this pull request to the merge queue May 26, 2026
Merged via the queue into main with commit 47f664d May 26, 2026
34 checks passed
@caalador caalador deleted the fix/2433-add-npm-excludePostinstallPackages branch May 26, 2026 06:37
vaadin-bot added a commit that referenced this pull request May 26, 2026
…24442)

This PR cherry-picks changes from the original PR #24421 to branch 25.1.
---
#### Original PR description
> Adds a configuration option to exclude packages from running
postinstall scripts. Mirrors the existing npm.postinstallPackages
add-list and applies to both the built-in default entries (e.g. esbuild,
@vaadin/vaadin-usage-statistics) and any packages added via
postinstallPackages. Wired through Options, the Maven and Gradle
plugins, and DevModeInitializer.
> 
> Refs #24333
>

Co-authored-by: Tomi Virtanen <tltv@vaadin.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants