Skip to content

feat: Only install npm packages that have been available more than a day#24338

Merged
Artur- merged 3 commits into
mainfrom
npm-install-delay-enforce
May 15, 2026
Merged

feat: Only install npm packages that have been available more than a day#24338
Artur- merged 3 commits into
mainfrom
npm-install-delay-enforce

Conversation

@Artur-
Copy link
Copy Markdown
Member

@Artur- Artur- commented May 14, 2026

Bumps the default from 0 to 1 to enable the supply-chain delay by default while keeping the window short enough that the same-day Vaadin upgrade pain is at least bounded. Users can still opt out by setting the option to 0.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 14, 2026

Test Results

 1 408 files  ±0   1 408 suites  ±0   1h 22m 43s ⏱️ -14s
10 150 tests ±0  10 080 ✅ ±0  70 💤 ±0  0 ❌ ±0 
10 625 runs  ±0  10 546 ✅ ±0  79 💤 ±0  0 ❌ ±0 

Results for commit ead2019. ± Comparison against base commit 371f6cd.

♻️ This comment has been updated with latest results.

@Artur- Artur- force-pushed the npm-install-delay-enforce branch from 8491e46 to b9f8611 Compare May 14, 2026 10:32
Bumps the default from 0 to 1 to enable the supply-chain delay by
default while keeping the window short enough that the same-day Vaadin
upgrade pain is at least bounded. Users can still opt out by setting
the option to 0.
@Artur- Artur- marked this pull request as ready for review May 14, 2026 13:33
@Artur- Artur- force-pushed the npm-install-delay-enforce branch from c61f566 to 41c48bb Compare May 14, 2026 13:33
@sonarqubecloud
Copy link
Copy Markdown

@Artur- Artur- requested a review from mcollovati May 15, 2026 10:31
@Artur- Artur- added this pull request to the merge queue May 15, 2026
Merged via the queue into main with commit 1311662 May 15, 2026
31 checks passed
@Artur- Artur- deleted the npm-install-delay-enforce branch May 15, 2026 11:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants