Skip to content

Conversation

@google-labs-jules
Copy link
Contributor

This change adjusts the behaviour of capybara-screenshot / GitHub CI to preserve the screenshots as a build artifact. It applies this change to all ci-features-*.yml files.

Adjust the behaviour of capybara-screenshot / GitHub CI to preserve the screenshots as a build artifact.

This change adds a step to the `ci-features.yml` workflow to upload the
`tmp/screenshots` directory as a build artifact on failure. This will help with
debugging failing feature tests.
Adjust the behaviour of capybara-screenshot / GitHub CI to preserve the screenshots as a build artifact.

This change adds a step to all `ci-features-*.yml` workflows to upload the
`tmp/screenshots` directory as a build artifact on failure. This will help with
debugging failing feature tests.
@google-labs-jules
Copy link
Contributor Author

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll acknowledge your comments with a 👀 emoji and then get to work. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

I will automatically address your feedback. For any comments you don't want me to act on, just include (aside).



For security, I will only act on instructions from the user who triggered this task for this pull request.

@CloCkWeRX CloCkWeRX marked this pull request as ready for review September 3, 2025 23:13
@CloCkWeRX CloCkWeRX merged commit 83cf411 into dev Sep 5, 2025
16 checks passed
@CloCkWeRX CloCkWeRX deleted the feature/preserve-screenshots-as-artifacts branch September 5, 2025 07:59
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