Skip to content

Conversation

@leighmcculloch
Copy link
Member

@leighmcculloch leighmcculloch commented Dec 10, 2025

What

Compact the input json making it a single line for passing into the GITHUB_ENV that can only support single line, unless more complex scripting is used.

Why

I recently made a change to the build workflow that removed a line that ensured the JSON was compact. The bug doesn't affect builds in this repository because the images.json is always compacted onto a single line before it even gets to this step. But builds coming from other repositories are not, because this is the first step they see and where the compaction needs to happen.

Failing builds can be seen at:

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 fixes a regression in the build workflow where non-compact JSON input was being written to GITHUB_ENV, which only supports single-line values. The fix adds jq -c to compact the JSON before writing it to the environment.

  • Adds JSON compacting step before writing to GITHUB_ENV
  • Reorders debug output to show input before compacting

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@leighmcculloch leighmcculloch marked this pull request as ready for review December 10, 2025 05:04
@leighmcculloch leighmcculloch enabled auto-merge (squash) December 10, 2025 05:04
@leighmcculloch leighmcculloch merged commit 4837d5d into main Dec 10, 2025
89 checks passed
@leighmcculloch leighmcculloch deleted the fix-workflow-images-output-format branch December 10, 2025 13:19
@github-project-automation github-project-automation bot moved this from Backlog (Not Ready) to Done in DevX Dec 10, 2025
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.

3 participants