Skip to content

feat: add npm.excludePostinstallPackages option#24421

Open
tltv wants to merge 2 commits into
mainfrom
fix/2433-add-npm-excludePostinstallPackages
Open

feat: add npm.excludePostinstallPackages option#24421
tltv wants to merge 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

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.

1 participant