Skip to content

Remove duplicate preview comment status - #302

Merged
arjunkomath merged 1 commit into
mainfrom
fix/preview-comment-duplicate-status
Aug 22, 2026
Merged

Remove duplicate preview comment status#302
arjunkomath merged 1 commit into
mainfrom
fix/preview-comment-duplicate-status

Conversation

@arjunkomath

Copy link
Copy Markdown
Member

Summary

  • omit redundant footer text from non-failure preview deployment comments
  • retain detailed failure descriptions
  • cover both behaviors in the existing preview deployment test

Verification

  • cd web && pnpm exec vitest run tests/preview-deployments.test.ts
  • cd web && ./node_modules/.bin/tsc --noEmit
  • cd web && pnpm exec oxfmt --check lib/preview-deployments.ts tests/preview-deployments.test.ts

@github-actions

Copy link
Copy Markdown

React Doctor found no new issues. 🎉

Reviewed by React Doctor for commit 0f8ef77.

@arjunkomath
arjunkomath added this pull request to the merge queueAug 22, 2026
Merged via the queue into main with commit cd04609Aug 22, 2026
7 checks passed
@arjunkomath
arjunkomath deleted the fix/preview-comment-duplicate-status branch August 22, 2026 01:35
Sign up for freeto 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

@arjunkomath