Skip to content

test: cover PR/MR platform helpers#426

Merged
backnotprop merged 1 commit intobacknotprop:mainfrom
sudorest:test/pr-364-platform-helper-coverage
Mar 29, 2026
Merged

test: cover PR/MR platform helpers#426
backnotprop merged 1 commit intobacknotprop:mainfrom
sudorest:test/pr-364-platform-helper-coverage

Conversation

@sudorest
Copy link
Copy Markdown
Contributor

Summary

  • add regression coverage for PR/MR URL parsing, labels, display helpers, and CLI selection across GitHub and GitLab
  • include self-hosted GitLab parsing coverage to protect the platform routing introduced in feat: GitLab merge request review support #364
  • migrate the review editor's open-PR preference to a platform-neutral storage key while preserving reads from the legacy GitHub-specific key

Verification

  • bun test packages/shared/pr-provider.test.ts packages/shared/review-core.test.ts

Context

Follow-up to #364 to protect the new GitHub/GitLab review abstraction with focused helper coverage and a small platform-neutral naming cleanup.

@backnotprop backnotprop merged commit 9b69d17 into backnotprop:main Mar 29, 2026
5 checks passed
stk-code pushed a commit to stk-code/plannotator that referenced this pull request Mar 31, 2026
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