Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .distignore
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@ composer.lock
node_modules/
package.json
package-lock.json
.npmrc

# WordPress
.wordpress-org
Expand Down
1 change: 1 addition & 0 deletions .npmrc
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
min-release-age=7

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Express the seven-day gate in seconds

npm's min-release-age documentation defines this value in seconds, so 7 delays eligible releases by only seven seconds rather than the seven days stated in the commit. This leaves the intended supply-chain protection effectively absent; use 604800 for seven days.

Useful? React with 👍 / 👎.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Pin npm to a version that supports the gate

Checked .github/workflows/pr-playground-preview.yml: the install job selects Node 20 and immediately runs npm ci, while Node 20 ships npm 10 (for example, the Node 20.19.5 release includes npm 10.8.2), whose config reference does not support min-release-age. In that workflow the option is treated as unknown and ignored, so pin or install a supporting npm version before relying on this control.

Useful? React with 👍 / 👎.

Loading