8000 docs: add hidden Google Analytics view pixel to README by adeelahmad · Pull Request #1 · adeelahmad/mlx-grpo-rl · GitHub
[go: up one dir, main page]

Skip to content

docs: add hidden Google Analytics view pixel to README - #1

Open
adeelahmad wants to merge 1 commit into
masterfrom
claude/visitor-badges-adkdjp
Open

adeelahmad wants to merge 1 commit into
masterfrom
claude/visitor-badges-adkdjp

Conversation

@adeelahmad
Copy link
Copy Markdown
Owner

Requested by Adeel · project thread

Before: no way to see how often the README is viewed.

After: a hidden 1x1 image at the end of the README sends a page_view to the existing Google Analytics property (G-RKFB16BPXJ) each time it loads, with the repo as page location and a repo parameter. Nothing visible changes.

How: a plain GA4 collect request as an image, no scripts or secrets. GitHub loads README images through its camo proxy, so hits carry no visitor details, all count as one visitor per repo, and Google may filter some as bot traffic.


Generated by Claude Code

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/c
8000
ode/session_01FyWTqq9D1G3eeNgHdBBhyQ
@adeelahmad
adeelahmad force-pushed the claude/visitor-badges-adkdjp branch from e42c411 to 28f07cd Compare October 20, 2026 02:37
@adeelahmad adeelahmad changed the title Add a hidden Google Analytics view pixel to the README docs: add hidden Google Analytics view pixel to README Oct 20, 2026
Copy link
Copy Markdown
Owner Author

Lint & Format, Type Check and the test jobs are failing on Python code this PR doesn't touch. Ruff reports 6 errors in existing source files, and this diff only adds one line to README.md. The failures come from the current master code, not this change. No fix exists yet, and fixing them here would widen a README-only PR, so it's safe to merge as is.


Generated by Claude Code

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

0