Skip to content

test: exercise CI with deliberate type error#2910

Open
k11kirky wants to merge 1 commit into
mainfrom
posthog-code/ci-test-break
Open

test: exercise CI with deliberate type error#2910
k11kirky wants to merge 1 commit into
mainfrom
posthog-code/ci-test-break

Conversation

@k11kirky

Copy link
Copy Markdown
Contributor

Test PR to verify the CI pipeline.

Changes

  • Added a test comment to the README.
  • Added a deliberate TypeScript type error in packages/host-router/src/ports/connectivity-client.ts so the typecheck job fails.

This is intentionally broken to confirm CI reports failures as expected. Do not merge — will be reverted/closed once CI behavior is confirmed.

🤖 Generated with Claude Code

Adds a test comment to the README and a deliberate TypeScript type error
in connectivity-client.ts so the typecheck job fails. Intended to verify
the CI pipeline reports failures as expected on this branch.

Generated-By: PostHog Code
Task-Id: e13040ec-b97f-4622-8500-e370c8dd6392
@github-actions

Copy link
Copy Markdown

React Doctor found no issues in the changed files. 🎉

Reviewed by React Doctor for commit 95eda54.

@greptile-apps

greptile-apps Bot commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Reviews (1): Last reviewed commit: "test: add README comment and deliberate ..." | Re-trigger Greptile

Comment thread README.md
Comment on lines +10 to +11
<!-- CI test comment: verifying the pipeline picks up changes on this branch -->

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 This HTML comment has no value for anyone reading the README and violates the "no superfluous parts" simplicity rule. It should be removed before any merge.

Suggested change
<!-- CI test comment: verifying the pipeline picks up changes on this branch -->

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

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