Skip to content

chore(docs): remove shrill.en.dev analytics script#457

Merged
jdx merged 1 commit intomainfrom
chore/remove-shrill-analytics
May 2, 2026
Merged

chore(docs): remove shrill.en.dev analytics script#457
jdx merged 1 commit intomainfrom
chore/remove-shrill-analytics

Conversation

@jdx
Copy link
Copy Markdown
Owner

@jdx jdx commented May 2, 2026

Removes the proxied shrill.en.dev analytics snippet from the VitePress docs config — the endpoint is going away.


Note

Low Risk
Low risk: removes a docs-site analytics script tag only, with no impact on application logic or data handling.

Overview
Removes the shrill.en.dev analytics script injection from the VitePress docs head config, stopping the docs site from loading and sending events to that endpoint.

Reviewed by Cursor Bugbot for commit 08592fc. Bugbot is set up for automated code reviews on this repo. Configure here.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request removes the analytics script configuration from the VitePress documentation site. I have no feedback to provide.

@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented May 2, 2026

Greptile Summary

This PR removes the shrill.en.dev analytics script from the VitePress docs configuration, as the endpoint is being decommissioned. The change is a clean deletion with no impact on docs functionality or build output.

Confidence Score: 5/5

Safe to merge — straightforward removal of a deprecated analytics snippet with no functional side effects.

Single-file change that deletes a dead external script reference; no logic, no security implications, no syntax issues.

No files require special attention.

Important Files Changed

Filename Overview
docs/.vitepress/config.mjs Removes the decommissioned shrill.en.dev analytics script tag from the VitePress head config; no issues found.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[VitePress Build] --> B[docs/.vitepress/config.mjs]
    B --> C[Head Tags]
    C --> D[Meta / Link tags]
    C -.->|removed| E["❌ shrill.en.dev analytics script"]
Loading

Reviews (1): Last reviewed commit: "chore(docs): remove shrill.en.dev analyt..." | Re-trigger Greptile

@jdx jdx merged commit d98447c into main May 2, 2026
18 checks passed
@jdx jdx deleted the chore/remove-shrill-analytics branch May 2, 2026 13:44
jdx pushed a commit that referenced this pull request May 2, 2026
### 🚜 Refactor

- extract providers and core types into fnox-core crate by
[@jdx](https://github.com/jdx) in
[#458](#458)

### 📚 Documentation

- prefix star count with ★ glyph and populate it on deploy by
[@jdx](https://github.com/jdx) in
[#447](#447)
- add favicons and web app manifest by [@jdx](https://github.com/jdx) in
[#448](#448)

### 🔍 Other Changes

- **(docs)** remove shrill.en.dev analytics script by
[@jdx](https://github.com/jdx) in
[#457](#457)
- **(release)** add musl Linux targets for Alpine compatibility by
[@jdx](https://github.com/jdx) in
[#452](#452)
- add plausible analytics by [@jdx](https://github.com/jdx) in
[#451](#451)
- bump hk to 1.44.3 by [@jdx](https://github.com/jdx) in
[#454](#454)

### 📦️ Dependency Updates

- update autofix-ci/action action to v1.3.4 by
[@renovate[bot]](https://github.com/renovate[bot]) in
[#455](#455)
- update apple-actions/import-codesign-certs action to v7 by
[@renovate[bot]](https://github.com/renovate[bot]) in
[#456](#456)
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