WARNING: THIS SITE IS A MIRROR OF GITHUB.COM / IT CANNOT LOGIN OR REGISTER ACCOUNTS / THE CONTENTS ARE PROVIDED AS-IS / THIS SITE ASSUMES NO RESPONSIBILITY FOR ANY DISPLAYED CONTENT OR LINKS / IF YOU FOUND SOMETHING MAY NOT GOOD FOR EVERYONE, CONTACT ADMIN AT ilovescratch@foxmail.com
Skip to content

Conversation

@caugner
Copy link
Contributor

@caugner caugner commented Dec 3, 2025

Description

Removes caporal from dependencies.

Motivation

It was never used (added in #26410, which already used @caporal/core), and not removed in #42210.

Additional details

Eliminates 2 low severity vulnerabilities reported by npm install --audit:

// Before
23 vulnerabilities (2 low, 3 moderate, 18 high)

// After
21 vulnerabilities (3 moderate, 18 high)

Related issues and pull requests

Follow-up of #42210.

@caugner caugner requested review from a team and mdn-bot as code owners December 3, 2025 11:47
@caugner caugner requested review from argl and removed request for a team December 3, 2025 11:47
@github-actions github-actions bot added system [PR only] Infrastructure and configuration for the project size/l [PR only] 501-1000 LoC changed labels Dec 3, 2025
@argl argl merged commit d068c8e into main Dec 3, 2025
18 checks passed
@argl argl deleted the remove-caporal branch December 3, 2025 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/l [PR only] 501-1000 LoC changed system [PR only] Infrastructure and configuration for the project

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants