Skip to content

chore: clarify E2E test comments - #1474

Merged
Kevinjohn merged 3 commits into
mainfrom
feature/issue-1469-comment-style
Oct 11, 2026
Merged

Kevinjohn merged 3 commits into
mainfrom
feature/issue-1469-comment-style

Conversation

@Kevinjohn

@Kevinjohn Kevinjohn commented Oct 11, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • Clarify E2E comments around browser fixture and test intent.
  • Let unexpected Playwright errors surface instead of converting them into false results.

Validation

  • pnpm run gate:all passed.
  • pnpm run e2e passed (294 tests).

Merge predecessor: none

Closes #1469

Merge order

#1472 → #1473, #1474, #1466

Merge predecessor for this PR: none.

Conflicts: none.

Signed-off-by: Kevinjohn Gallagher <kevinjohngallagher@hotmail.com>
Signed-off-by: Kevinjohn Gallagher <kevinjohngallagher@hotmail.com>
@Kevinjohn

Copy link
Copy Markdown
Owner Author

Polish review: no P1 or P2 findings, no changes pushed.

Merge predecessor: none.

@Kevinjohn Kevinjohn added polish-done Polish review complete and removed polish-in-progress Polish review running labels Oct 11, 2026
@Kevinjohn
Kevinjohn merged commit 6a1e584 into main Oct 11, 2026
5 checks passed
@Kevinjohn
Kevinjohn deleted the feature/issue-1469-comment-style branch October 11, 2026 18:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

polish-done Polish review complete

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Align E2E and documentation source comments with the house style

1 participant