Skip to content

Conversation

@SimonWahlin
Copy link
Member

No description provided.

Copilot AI review requested due to automatic review settings July 6, 2025 11:43
@SimonWahlin SimonWahlin merged commit 89b1e75 into main Jul 6, 2025
1 check failed
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 adds logging of the module version in the build action to surface which version is being packed.

  • Logs the ModuleVersion environment variable before running the pack task.
  • Updates the run step in .github/actions/build/action.yml to include verbose output.
Comments suppressed due to low confidence (3)

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

  • [nitpick] Consider using a YAML multiline script block (|) for the run step to improve readability and reduce the complexity of a single long inline command.
    run: $ModuleVersion=$Env:ModuleVersion;Write-Verbose "Using version: $ModuleVersion" -Verbose;./build.ps1 -tasks pack

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

  • Write-Verbose output may be suppressed unless -Verbose is enabled; using Write-Host or explicitly setting $VerbosePreference = 'Continue' can ensure the version is always visible in the GitHub Actions log.
    run: $ModuleVersion=$Env:ModuleVersion;Write-Verbose "Using version: $ModuleVersion" -Verbose;./build.ps1 -tasks pack

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

  • [nitpick] The temporary assignment to $ModuleVersion isn’t strictly needed; you can reference $Env:ModuleVersion directly in the logging command to simplify the script.
    run: $ModuleVersion=$Env:ModuleVersion;Write-Verbose "Using version: $ModuleVersion" -Verbose;./build.ps1 -tasks pack

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