Skip to content

Conversation

@MariusStorhaug
Copy link
Member

Description

This pull request makes a minor change to the ConvertTo-GitHubOutput function by removing the -Compress parameter from ConvertTo-Json calls. This pretty-prints the JSON output, so that its nicer to review in GitHub Actions logs.

  • Simplified JSON output:
    • Removed the -Compress parameter from ConvertTo-Json calls for both valid JSON strings and non-string values. This ensures consistent formatting without compression.

Type of change

  • 📖 [Docs]
  • 🪲 [Fix]
  • 🩹 [Patch]
  • ⚠️ [Security fix]
  • 🚀 [Feature]
  • 🌟 [Breaking change]

Checklist

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas

@MariusStorhaug MariusStorhaug self-assigned this Jun 6, 2025
Copilot AI review requested due to automatic review settings June 6, 2025 16:47
@MariusStorhaug MariusStorhaug requested a review from a team as a code owner June 6, 2025 16:47
Copy link
Contributor

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 ConvertTo-GitHubOutput function to pretty-print JSON output, improving readability in GitHub Actions logs.

  • Removed the -Compress parameter from ConvertTo-Json calls for valid JSON strings.
  • Removed the -Compress parameter from ConvertTo-Json calls for non-string values.

@MariusStorhaug MariusStorhaug merged commit a7c6c7d into main Jun 6, 2025
10 checks passed
@MariusStorhaug MariusStorhaug deleted the non-compress branch June 6, 2025 18:34
@github-project-automation github-project-automation bot moved this from Todo to Done in GitHub PowerShell Module Jun 6, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Jun 6, 2025

Module GitHub - 0.29.1 published to the PowerShell Gallery.

@github-actions
Copy link
Contributor

github-actions bot commented Jun 6, 2025

GitHub release for GitHub v0.29.1 has been created.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant