Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

build(server,common): Upgrade mocha to latest minor (10.8.2) #23980

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

tylerbutler
Copy link
Member

Upgrades mocha to latest minor (10.8.2)

@github-actions github-actions bot added area: build Build related issues area: server Server related issues (routerlicious) base: main PRs targeted against main branch labels Mar 4, 2025
@tylerbutler tylerbutler marked this pull request as ready for review March 5, 2025 01:41
@tylerbutler tylerbutler requested review from Copilot and a team March 5, 2025 01:41

Choose a reason for hiding this comment

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

PR Overview

This PR upgrades Mocha from its previous versions (10.4.0/10.2.0) to the latest minor version 10.8.2 across multiple lockfiles, ensuring that dependent packages and peer dependency references are updated accordingly.

  • Updated Mocha version and its references in lockfiles for both common and server packages.
  • Adjusted ancillary dependency versions (e.g., diff, ansi-colors, debug, workerpool, yargs-parser) to align with the Mocha upgrade.
  • In the common lib, an additional dependency (encoding) was added for build-cli, which also appears in associated dependencies.

Reviewed Changes

File Description
common/build/eslint-plugin-fluid/pnpm-lock.yaml Upgraded Mocha and adjusted related dependency versions (e.g., mocha-multi-reporters, diff, ansi‑colors)
server/routerlicious/pnpm-lock.yaml Updated Mocha version from 10.2.0 to 10.8.2 and simplified debug version references
common/lib/common-utils/pnpm-lock.yaml Upgraded Mocha to 10.8.2 and introduced an extra dependency (encoding) in the build-cli version specification

Copilot reviewed 20 out of 20 changed files in this pull request and generated no comments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: build Build related issues area: server Server related issues (routerlicious) base: main PRs targeted against main branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant