Skip to content

Conversation

@shati-patel
Copy link
Contributor

Updates the jest version, which should unblock #4174. (I'm not sure why dependabot hasn't suggested this update yet—possibly because we were on a weird alpha version of jest previously.)

Anyway, see #4174 (comment) for the details!


(ℹ️ Note: I don't plan to merge this until after the GH Universe code freeze is lifted 🧊)

@shati-patel shati-patel marked this pull request as ready for review October 27, 2025 15:43
@shati-patel shati-patel requested a review from a team as a code owner October 27, 2025 15:43
Copilot AI review requested due to automatic review settings October 27, 2025 15:43
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates Jest dependencies from an alpha version (30.0.0-alpha.7) to a stable release (30.2.0), which is needed to unblock another pull request. The update moves away from pre-release versions to a production-ready version of Jest.

  • Upgraded @jest/environment from v30.0.0-alpha.7 to v30.2.0
  • Upgraded @jest/environment-jsdom-abstract from v30.0.0-alpha.7 to v30.2.0
Files not reviewed (1)
  • extensions/ql-vscode/package-lock.json: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

@robertbrignull robertbrignull left a comment

Choose a reason for hiding this comment

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

LGTM. Makes sense to me to do this upgrade and get back onto normal releases if we can.

Copy link
Member

@koesie10 koesie10 left a comment

Choose a reason for hiding this comment

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

LGTM, although I think it would also make sense to update the version of the jest package itself.

Copy link
Member

@koesie10 koesie10 left a comment

Choose a reason for hiding this comment

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

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants