Skip to content

fix(site): posthog site cleanup#7776

Open
mhartington wants to merge 1 commit intomainfrom
posthog-cleanup
Open

fix(site): posthog site cleanup#7776
mhartington wants to merge 1 commit intomainfrom
posthog-cleanup

Conversation

@mhartington
Copy link
Copy Markdown
Member

@mhartington mhartington commented Apr 6, 2026

Summary by CodeRabbit

  • Chores
    • Updated internal site configuration identifier.

@mhartington mhartington requested a review from carlagn April 6, 2026 13:53
@vercel
Copy link
Copy Markdown

vercel bot commented Apr 6, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
blog Ready Ready Preview, Comment Apr 6, 2026 1:55pm
docs Ready Ready Preview, Comment Apr 6, 2026 1:55pm
eclipse Ready Ready Preview, Comment Apr 6, 2026 1:55pm
site Ready Ready Preview, Comment Apr 6, 2026 1:55pm

Request Review

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 6, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 52fb8c26-1966-452c-90d5-69f07a92b344

📥 Commits

Reviewing files that changed from the base of the PR and between 67daddc and 240b790.

📒 Files selected for processing (1)
  • apps/site/src/instrumentation-client.ts

Walkthrough

The PostHog instrumentation client configuration updates the site name identifier from "mono-blog" to "mono-site" during initialization. This is a direct property value change with no modifications to control flow or error handling logic.

Changes

Cohort / File(s) Summary
PostHog Configuration
apps/site/src/instrumentation-client.ts
Updated the site_name property in PostHog's posthog.register call from "mono-blog" to "mono-site", reflecting a naming convention alignment for the application identifier.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive The title refers to a real change in the PR (PostHog configuration update) but uses vague language ('cleanup') that doesn't clearly convey what was actually fixed. Consider a more specific title like 'fix(site): update posthog site_name to mono-site' to clearly describe the actual change being made.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@argos-ci
Copy link
Copy Markdown

argos-ci bot commented Apr 6, 2026

The latest updates on your projects. Learn more about Argos notifications ↗︎

Build Status Details Updated (UTC)
default (Inspect) ✅ No changes detected - Apr 6, 2026, 2:02 PM

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