Skip to content

Conversation

andyleejordan
Copy link
Member

Hopefully we can get a release out soon.

@Copilot Copilot AI review requested due to automatic review settings September 5, 2025 17:58
@andyleejordan andyleejordan requested a review from a team as a code owner September 5, 2025 17:58
Copy link
Contributor

@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 contains OneBranch-related fixes aimed at enabling a release. The changes include increasing test timeout values and adding a scheduled CodeQL scan to improve the build pipeline reliability.

  • Increased diagnostic timeout from 30 to 60 seconds (doubling the wait time)
  • Added weekly CodeQL security scanning to the OneBranch pipeline

Reviewed Changes

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

File Description
test/PowerShellEditorServices.Test.E2E/LanguageServerProtocolMessageTests.cs Increased timeout for diagnostic waiting from 30 to 60 seconds
.pipelines/PowerShellEditorServices-OneBranch.yml Added weekly CodeQL schedule configuration

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

@andyleejordan andyleejordan enabled auto-merge (rebase) September 5, 2025 18:01
Since there is no way to do this in the UI,
but we also went down to one pipeline.
Copy link
Collaborator

@SeeminglyScience SeeminglyScience left a comment

Choose a reason for hiding this comment

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

LGTM!

@andyleejordan andyleejordan merged commit 2b24392 into main Sep 5, 2025
9 checks passed
@andyleejordan andyleejordan deleted the slow-test branch September 5, 2025 19:00
@andyleejordan andyleejordan added the Ignore Exclude from the changelog. label Sep 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ignore Exclude from the changelog.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants