Skip to content

Conversation

@SimonWahlin
Copy link
Member

No description provided.

Copilot AI review requested due to automatic review settings July 6, 2025 11:46
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 the build action to wrap the PowerShell command in a multiline script, explicitly capture the module version from the environment, and add a log statement for that version.

  • Convert single-line run into a multiline PowerShell script
  • Assign and log ModuleVersion before packaging
  • Preserve original ./build.ps1 -tasks pack invocation
Comments suppressed due to low confidence (2)

.github/actions/build/action.yml:51

  • Write-Verbose messages are hidden by default in GitHub Actions logs unless -Verbose is enabled; consider using Write-Host for consistent output of the module version.
      Write-Verbose "Using version: $ModuleVersion" -Verbose

.github/actions/build/action.yml:50

  • [nitpick] Assigning $Env:ModuleVersion to a local variable is redundant; you can reference $Env:ModuleVersion directly to simplify the script.
      $ModuleVersion = $Env:ModuleVersion

@github-actions
Copy link
Contributor

github-actions bot commented Jul 6, 2025

Linux Test Results

 1 files  16 suites   4s ⏱️
71 tests 70 ✅ 0 💤 1 ❌
72 runs  71 ✅ 0 💤 1 ❌

For more details on these failures, see this check.

Results for commit 0800f9d.

@github-actions
Copy link
Contributor

github-actions bot commented Jul 6, 2025

Win Test Results

 1 files  16 suites   6s ⏱️
71 tests 70 ✅ 0 💤 1 ❌
72 runs  71 ✅ 0 💤 1 ❌

For more details on these failures, see this check.

Results for commit 0800f9d.

@SimonWahlin SimonWahlin merged commit 8790ddf into main Jul 6, 2025
3 of 7 checks passed
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.

2 participants