Skip to content

Conversation

@pwilczynskiclearcode
Copy link
Contributor

No description provided.

Copilot AI review requested due to automatic review settings April 8, 2025 14:58
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.

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

.github/workflows/ai-runner-models.yaml:138

  • [nitpick] Consider reviewing the YAML formatting for the 'outputs' setting; using a mapping format instead of a comma-separated string may improve clarity and ensure correct parsing.
outputs: type=image,compression=uncompressed

context: .
platforms: linux/amd64
push: ${{ github.event_name != 'pull_request' }}
push: true
Copy link

Copilot AI Apr 8, 2025

Choose a reason for hiding this comment

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

Unconditionally setting 'push' to true may lead to images being pushed on pull request events. Please verify that this behavior is intended to avoid unintended deployments.

Suggested change
push: true
push: ${{ github.event_name == 'push' && github.ref == 'refs/heads/main' }}

Copilot uses AI. Check for mistakes.
@pwilczynskiclearcode pwilczynskiclearcode force-pushed the models-without-compression branch 6 times, most recently from 0212381 to fd1ea32 Compare April 9, 2025 14:05
@pwilczynskiclearcode pwilczynskiclearcode force-pushed the models-without-compression branch from fd1ea32 to e373f43 Compare April 9, 2025 14:39
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