Skip to content

setup: add image-gen boolean input to conditionally install image tools#4

Merged
mmathieum merged 3 commits into
masterfrom
copilot/add-image-gen-boolean-input
Apr 15, 2026
Merged

setup: add image-gen boolean input to conditionally install image tools#4
mmathieum merged 3 commits into
masterfrom
copilot/add-image-gen-boolean-input

Conversation

Copilot AI commented Apr 15, 2026

Copy link
Copy Markdown
Contributor
  • Add image-gen boolean input to conditionally install image tools
  • Add sudo apt-get update before apt-get install in the image dependencies step for consistency and robustness
  • Remove standalone MT set MT_GENERATE_IMAGES env step; move MT_GENERATE_IMAGES as a step-level env on MT code setup only

Copilot AI changed the title [WIP] Add support for image-gen boolean input in setup action setup: add image-gen boolean input to conditionally install image tools Apr 15, 2026
Copilot AI requested a review from mmathieum April 15, 2026 13:25
Comment thread .github/actions/setup/action.yml
Copilot AI requested a review from mmathieum April 15, 2026 13:29
@mmathieum mmathieum marked this pull request as ready for review April 15, 2026 13:29
@mmathieum

Copy link
Copy Markdown
Member

/gemini review

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request introduces an image-gen input to the setup action, allowing for conditional installation of image-related dependencies (inkscape and imagemagick). The feedback suggests adding sudo apt-get update to the new installation step for better reliability and simplifying the environment variable assignment by leveraging the boolean nature of the input.

Comment thread .github/actions/setup/action.yml Outdated
Comment thread .github/actions/setup/action.yml Outdated
Comment thread .github/actions/setup/action.yml Outdated
Comment thread .github/actions/setup/action.yml Outdated
…S to code setup step

Agent-Logs-Url: https://github.com/mtransitapps/gh-actions/sessions/22614c4e-de3b-4cb3-8484-d7371b17e136

Co-authored-by: mmathieum <177998+mmathieum@users.noreply.github.com>
Copilot AI requested a review from mmathieum April 15, 2026 14:48
@mmathieum mmathieum merged commit 878f2d9 into master Apr 15, 2026
@mmathieum mmathieum deleted the copilot/add-image-gen-boolean-input branch April 15, 2026 14:53
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.

Action setup > support for "image-gen" boolean input

2 participants