Skip to content

refactor: enforce screenshot regeneration #4528

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.

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

Merged
merged 12 commits into from
Jul 15, 2025

Conversation

mfranzke
Copy link
Collaborator

Proposed changes

enforce regenerating all screenshots.

Types of changes

  • Bugfix (non-breaking change that fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Refactoring (improvements to existing components or architectural decisions)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation Update (if none of the other choices apply)

Further comments

@mfranzke mfranzke self-assigned this Jul 14, 2025
@mfranzke mfranzke requested a review from nmerget as a code owner July 14, 2025 08:01
@mfranzke mfranzke added the 🍄🆙improvement New feature or request label Jul 14, 2025
Copy link
Contributor

@github-actions github-actions bot added the 📺showcases Changes to 1-n showcases label Jul 14, 2025
@github-actions github-actions bot removed the 📺showcases Changes to 1-n showcases label Jul 14, 2025
github-actions bot and others added 3 commits July 14, 2025 09:52
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@nmerget
Copy link
Collaborator

nmerget commented Jul 15, 2025

Why was this required?

@mfranzke
Copy link
Collaborator Author

Why was this required?

because we do have a lot of changes within #4283 that aren't based on the CSS changes being made within that PR. Looked like that our screenshots were outdated.

nmerget
nmerget previously approved these changes Jul 15, 2025
github-actions bot and others added 2 commits July 15, 2025 08:26
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@mfranzke mfranzke merged commit 89c49f8 into main Jul 15, 2025
7 checks passed
@mfranzke mfranzke deleted the refactor-enforce-screenshot-regeneration branch July 15, 2025 14:07
mfranzke added a commit that referenced this pull request Jul 16, 2025
* refactor: enforce screenshot regeneration

* auto update snapshots (#4529)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* refactor: enforce screenshot regeneration

* Revert "refactor: enforce screenshot regeneration"

This reverts commit 56a69d5.

* Revert "refactor: enforce screenshot regeneration"

This reverts commit 910608b.

* auto update snapshots (#4530)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* test: updated snapshots

* test: updated snapshots

* refactor: removed all snapshots to enforce their regeneration

* auto update snapshots (#4539)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🍄🆙improvement New feature or request
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

2 participants