Skip to content

chore: bump email-template-workflows to v3 - #68

Merged
atherdon merged 2 commits into
mainfrom
chore/bump-email-template-workflows-v3
Nov 3, 2026
Merged

atherdon merged 2 commits into
mainfrom
chore/bump-email-template-workflows-v3

Conversation

@atherdon

@atherdon atherdon commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

Summary

Bump LLazyEmail/email-template-workflows to v3 (breaking).

  • No @v3 tag/release exists yet on email-template-workflows (PR feat: add xhtmlDocument() with baked-in XHTML shell #51 merged to main as 205c017968bd23a59cb4ccbc7d436fccfed11e57). This PR pins that SHA until @v3 is published.
  • v3 switches defaults to the generic contract: product-name=files, size-limit-kb=0, artifact-name=rendered. Builds also get FILE_INPUT / FILE_OUTPUT (EMAIL_TEMPLATE_* still set for compatibility).

Migration applied

  • Pin action to 205c017… with a # v3 comment.
  • Set product-name / size-limit-kb explicitly where email behavior should be preserved (or accept the new generic defaults for sandbox HTML).
  • No build-command / EMAIL_TEMPLATE_* usage in this workflow, so no FILE_* renames were required.

Test plan

  • CI on this PR is green
  • Render / sandbox job still uploads the artifact and (when enabled) posts/updates the PR comment
  • After @v3 is tagged, optionally retarget the pin to @v3 or the release SHA

Do not merge until CI is reviewed.

Pin LLazyEmail/email-template-workflows@205c017 (v3 on main; no @V3 tag yet) and apply the v3 migration so defaults keep working.
@github-actions

github-actions Bot commented Nov 3, 2026 •

Copy link
Copy Markdown

Rendered emails

This PR rendered emails.

Open the run → Artifacts → download → open index.html.

Replace the temporary SHA pin with the published floating major tag now that v3.0.0 / @V3 exist.
@atherdon
atherdon merged commit c494594 into main Nov 3, 2026
8 checks passed
@atherdon
atherdon deleted the chore/bump-email-template-workflows-v3 branch November 3, 2026 10:52
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.

1 participant