Skip to content

refactor: extract default snapshot options for visual tests#7462

Merged
laske185 merged 6 commits intodevelopfrom
7460-v3
Mar 14, 2025
Merged

refactor: extract default snapshot options for visual tests#7462
laske185 merged 6 commits intodevelopfrom
7460-v3

Conversation

@deleonio
Copy link
Copy Markdown
Contributor

@deleonio deleonio commented Mar 12, 2025

This pull request includes several updates to the GitHub Actions workflows and some improvements to the visual tests. The most important changes involve updating Node.js and pnpm versions across multiple workflow files, adding new input parameters for snapshot updates, and refactoring the visual tests to use default snapshot options.

Updates to GitHub Actions workflows:

Enhancements to snapshot update workflow:

Improvements to visual tests:

Refs: #7460

The A11y and PO reviews will only take place after all other DoD steps have been completed by the Developer:

  • Meaningful pull request title for the release notes
  • Pull request is linked to an issue and all changes relate to the issue
  • Tests to protect this code implemented (if applicable)
  • Manual test performed successfully (if applicable)
  • Documentation or migration has been updated (if applicable)

@deleonio deleonio requested review from laske185 and sdvg March 12, 2025 04:24
@deleonio deleonio added the v3 label Mar 12, 2025
@deleonio deleonio changed the title Refactor snapshot options to use constants for consistency in visual tests refactor: extract default snapshot options for visual tests Mar 12, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 12, 2025

@laske185 laske185 merged commit be81bca into develop Mar 14, 2025
9 checks passed
@laske185 laske185 deleted the 7460-v3 branch March 14, 2025 10:37
@publicuibot publicuibot bot locked and limited conversation to collaborators Mar 14, 2025
@deleonio deleonio removed the v3 label Jun 11, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Snapshot-Ratio 0.0,-Delete optional, -AXE off, Node-Version 22 and PNPM-Version 10

3 participants