Conversation

andyleejordan

We can rely on the pre-installed PSResourceGet and we can re-enable tests!

@JustinGrote seriously we can finally run tests again in the release pipeline 🤞

We can rely on the pre-installed PSResourceGet and we can re-enable tests!
@CopilotCopilot AI review requested due to automatic review settings February 26, 2025 19:07
@andyleejordanandyleejordan requested a review from a team as a code owner February 26, 2025 19:07
@andyleejordanandyleejordan enabled auto-merge (squash) February 26, 2025 19:07

Choose a reason for hiding this comment

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

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

Comments suppressed due to low confidence (2)

.pipelines/PowerShellEditorServices-Official.yml:91

  • Verify that the installPSResources.ps1 script now encapsulates all needed actions (registration of the PS repository and installation of PSResourceGet), since the explicit commands have been removed.
-          - pwsh: ./tools/installPSResources.ps1 -PSRepository CFS

.pipelines/PowerShellEditorServices-Official.yml:93

  • [nitpick] Consider updating the displayName accompanying this step so that it accurately reflects the action being performed, given the change from 'Build' to 'TestFull'.
 +          - pwsh: Invoke-Build TestFull -Configuration $(BuildConfiguration) -PSRepository CFS

@andyleejordanandyleejordan added the IgnoreExclude from the changelog.label Feb 26, 2025

Choose a reason for hiding this comment

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

LGTM!

@andyleejordanandyleejordan merged commit 1ee7adb into main Feb 26, 2025
8 checks passed
@andyleejordanandyleejordan deleted the onebranch branch February 26, 2025 19:18
Sign up for free to join this conversation on . Already have an account? Sign in to comment
IgnoreExclude from the changelog.
None yet

Successfully merging this pull request may close these issues.