Skip to content

Conversation

@andyleejordan
Copy link
Member

@andyleejordan andyleejordan commented Nov 21, 2025

Some TLC.

@andyleejordan andyleejordan marked this pull request as ready for review November 25, 2025 22:42
@andyleejordan andyleejordan requested a review from a team as a code owner November 25, 2025 22:42
Copilot AI review requested due to automatic review settings November 25, 2025 22:42
@andyleejordan andyleejordan changed the title Updates Update packages etc. Nov 25, 2025
Copilot finished reviewing on behalf of andyleejordan November 25, 2025 22:44
Copy link

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 various dependencies and makes configuration changes to the PowerShell extension. However, it contains several critical issues that will prevent the code from building or running correctly.

Key changes:

  • Dependency version updates for packages like @vscode/extension-telemetry, semver, and others
  • Modified ESLint configuration to use a new config helper
  • Updated Node.js version requirements and documentation

Reviewed changes

Copilot reviewed 10 out of 11 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
package.json Updates multiple dependency versions including critical production dependencies
package-lock.json Reflects the dependency version changes from package.json
src/extension.ts Changes TelemetryReporter import from default to named import
src/features/OpenInISE.ts Adds unnecessary type assertions to string concatenations
test/core/platform.test.ts Changes non-null assertion to type assertion for environment variable
eslint.config.mjs Introduces invalid import and replaces config function
CodeQL.yml Adds new CodeQL configuration file to classify library paths
.npmrc Updates comments and removes always-auth setting
.github/workflows/ci-test.yml Updates Node.js version from 20 to 22
docs/development.md Updates Node.js version requirements and links to Electron/Node versions
README.md Updates tested OS versions in CI

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

@andyleejordan andyleejordan merged commit a700e61 into main Nov 25, 2025
8 checks passed
@andyleejordan andyleejordan deleted the updates branch November 25, 2025 23:07
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.

3 participants