New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix snapshot tests job on CI #1096

Merged

ZacSweers merged 2 commits into main from z/fixSnapshotReports

Nov 15, 2024

Collaborator

ZacSweers commented Nov 14, 2024

Fix location of roborazzi outputs (was previously just pointing at the the main sources rather than where reports)
Fix the reports file name to disambiguate from the usual job

ZacSweers added 2 commits

November 14, 2024 15:57


          Ensure snapshot reports are a unique name

0c44625


          Fix output locations

d2f954d

ZacSweers requested a review from kateliu20

November 14, 2024 21:02

kateliu20 approved these changes

View reviewed changes

ZacSweers added this pull request to the merge queue

Merged via the queue into main with commit df11f7d

4 checks passed

ZacSweers deleted the z/fixSnapshotReports branch

November 15, 2024 05:08

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

Labels

None yet