Skip to content

Conversation

gagik
Copy link
Contributor

@gagik gagik commented Aug 26, 2025

Stacked on #2523

@gagik gagik marked this pull request as ready for review August 26, 2025 09:02
@Copilot Copilot AI review requested due to automatic review settings August 26, 2025 09:02
@gagik gagik requested a review from a team as a code owner August 26, 2025 09:02
Copy link

@Copilot 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 skips sort update and replace tests when the API is in strict mode and pins the MongoDB server version to 8.0.12 for crypt library downloads.

  • Adds a skip condition for sort-related update/replace tests in strict API mode
  • Updates MongoDB version specification from 'stable' to '8.0.12' for crypt library downloads

Reviewed Changes

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

File Description
packages/shell-api/src/integration.spec.ts Adds skipIfApiStrict() call to skip sort update/replace tests in strict mode
packages/build/src/packaging/download-crypt-library.ts Pins MongoDB version to 8.0.12 instead of using 'stable' version

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@gagik gagik added the no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion) label Aug 26, 2025
@gagik gagik changed the base branch from main to gagik/version-test August 26, 2025 09:23
Base automatically changed from gagik/version-test to main August 26, 2025 09:29
@gagik gagik merged commit e7eb1de into main Aug 26, 2025
111 of 117 checks passed
@gagik gagik deleted the gagik/api_strict branch August 26, 2025 09:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants