Skip to content

Conversation

@xsahil03x
Copy link
Member

@xsahil03x xsahil03x commented May 14, 2025

Description

This pull request updates the .github/workflows/update_goldens.yml workflow to use the Stream Bot to commit the updated golden files which fixes an issue where the commit was not triggering any workflow checks. Additionally it also improves the workflow by only committing the golden files.

@xsahil03x xsahil03x force-pushed the chore/use-stream-bot-in-golden-workflow branch from efcbda4 to eff1dbe Compare May 14, 2025 10:57
@xsahil03x xsahil03x force-pushed the chore/use-stream-bot-in-golden-workflow branch from 91d5f61 to 2e4b419 Compare May 14, 2025 11:58
@xsahil03x xsahil03x force-pushed the chore/use-stream-bot-in-golden-workflow branch from 55c1691 to 7aa59b1 Compare May 14, 2025 12:05
@xsahil03x xsahil03x force-pushed the chore/use-stream-bot-in-golden-workflow branch from 2ca978c to 85a626d Compare May 14, 2025 12:16
@xsahil03x xsahil03x requested a review from Copilot May 14, 2025 12:41
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 golden workflow to use the Stream SDK Bot for committing changes.

  • Added SSH key injection for secure repository checkout.
  • Updated git-auto-commit-action configuration to use Stream SDK Bot and a specific file pattern when committing golden image updates.
Comments suppressed due to low confidence (1)

.github/workflows/update_goldens.yml:36

  • Double-check that the file_pattern is comprehensive enough to capture all intended golden images. If golden images exist outside the specified test folder, consider a broader pattern to ensure all are committed.
          file_pattern: "**/test/**/goldens/*.png"

@xsahil03x xsahil03x merged commit 30c106e into master May 14, 2025
9 checks passed
@xsahil03x xsahil03x deleted the chore/use-stream-bot-in-golden-workflow branch May 14, 2025 12:47
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.

3 participants