Skip to content

Conversation

@nirinchev
Copy link
Contributor

No description provided.

Copilot AI review requested due to automatic review settings August 21, 2025 10:05
@nirinchev nirinchev requested a review from a team as a code owner August 21, 2025 10:05
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 dependencies across the project, including major version bumps for testing libraries, Node.js types, and various development tools. The changes also include type safety improvements and compatibility fixes for updated dependencies.

  • Updates numerous dependencies including @babel packages, testing libraries (chai, sinon, mocha), and VS Code types
  • Adds type casts to resolve compatibility issues with stricter type checking from updated dependencies
  • Modernizes icon path handling to use vscode.Uri instead of string paths
  • Updates glob import syntax and test runner implementation for newer API compatibility

Reviewed Changes

Copilot reviewed 34 out of 35 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
package.json Major dependency updates including TypeScript 5.x, Node 24.x types, and updated testing libraries
src/test/suite/index.ts Updates glob import and modernizes test runner implementation
src/explorer/*.ts Converts icon paths from strings to vscode.Uri objects
src/test/suite/**/*.test.ts Adds type casts and updates test mocks for compatibility with stricter types
scripts/update-grammar.ts Updates mkdirp import syntax
.gitattributes Adds package-lock.json handling attributes

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

@nirinchev nirinchev added the no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion) label Aug 21, 2025
@nirinchev nirinchev closed this Aug 21, 2025
@nirinchev nirinchev deleted the ni/dependencies branch August 21, 2025 10:40
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