Skip to content

fix(email-renderer): Temporarily remove snapshots to unblock main#19894

Merged
nshirley merged 1 commit intomainfrom
temp-disable-renderer-snapshots
Jan 15, 2026
Merged

fix(email-renderer): Temporarily remove snapshots to unblock main#19894
nshirley merged 1 commit intomainfrom
temp-disable-renderer-snapshots

Conversation

@nshirley
Copy link
Copy Markdown
Contributor

@nshirley nshirley commented Jan 15, 2026

Because:

  • We are seeing occasional snapshot failures only in CI for email-renderer

This Commit:

  • Skips and removes the snapshots to unblock main

This is temporary to just unblock main since we're seeing these failing occasionally on CI but not locally.

Unfortunately, we have to remove the full snapshot file since Jest will mark a test as failed if there is an associated snapshot but no tests checking against it. I currently have the ticket to fix this, so they'll be back soon!

Checklist

Put an x in the boxes that apply

  • My commit is GPG signed.
  • If applicable, I have modified or added tests which pass locally.
  • I have added necessary documentation (if appropriate).
  • I have verified that my changes render correctly in RTL (if appropriate).

Screenshots (Optional)

Please attach the screenshots of the changes made in case of change in user interface.

Other information (Optional)

Any other information that is important to this pull request.

Because:
 - We are seeing occasional snapshot failures only in CI for
   email-renderer

This Commit:
 - Skips and removes the snapshots to unblock main
@nshirley nshirley requested a review from a team as a code owner January 15, 2026 15:56
@nshirley nshirley merged commit d822cdb into main Jan 15, 2026
20 checks passed
@nshirley nshirley deleted the temp-disable-renderer-snapshots branch January 15, 2026 17:36
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