chore(docs): remove shrill.en.dev analytics script#614
Merged
Conversation
Greptile SummaryRemoves the proxied analytics Confidence Score: 5/5Safe to merge — trivial one-line removal with no side effects. Single-element removal from a static config array; no logic, no imports, no tests affected. No issues found. No files require special attention. Important Files Changed
Sequence DiagramsequenceDiagram
participant Browser
participant DocsPage as usage.jdx.dev
participant Analytics as shrill.en.dev (removed)
Note over Browser, Analytics: Before this PR
Browser->>DocsPage: Load page
DocsPage-->>Browser: HTML (includes analytics script tag)
Browser->>Analytics: POST /f5f1/event (pageview)
Analytics-->>Browser: 200 OK
Note over Browser, Analytics: After this PR
Browser->>DocsPage: Load page
DocsPage-->>Browser: HTML (no analytics script tag)
Reviews (1): Last reviewed commit: "chore(docs): remove shrill.en.dev analyt..." | Re-trigger Greptile |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #614 +/- ##
=======================================
Coverage 79.03% 79.03%
=======================================
Files 48 48
Lines 7235 7235
Branches 7235 7235
=======================================
Hits 5718 5718
Misses 1140 1140
Partials 377 377 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Contributor
There was a problem hiding this comment.
Code Review
This pull request removes a script tag and a meta tag from the VitePress configuration. A suggestion was made to include a trailing comma on the final array element to minimize future diff noise and align with TypeScript style conventions.
| src: "https://shrill.en.dev/shrill/script.js" | ||
| } | ||
| ] | ||
| ["meta", { name: "twitter:image", content: "https://usage.jdx.dev/android-chrome-512x512.png" }] |
Contributor
There was a problem hiding this comment.
It is recommended to maintain the trailing comma for the last element in the array when it is followed by a newline. This practice improves maintainability by reducing diff noise in future changes and adheres to common TypeScript style guides, such as the Google TypeScript Style Guide.
["meta", { name: "twitter:image", content: "https://usage.jdx.dev/android-chrome-512x512.png" }],
References
- The Google TypeScript Style Guide recommends using trailing commas in container literals (such as object literals and array literals) whenever there is a line break between the final element and the closing bracket to minimize diff noise. (link)
Merged
jdx
pushed a commit
that referenced
this pull request
May 3, 2026
### 🚀 Features - **(complete)** auto-completion for usage shebang scripts by [@jdx](https://github.com/jdx) in [#620](#620) ### 🐛 Bug Fixes - **(docs)** stack banner and pin close button on mobile by [@jdx](https://github.com/jdx) in [#603](#603) ### 📚 Documentation - **(site)** show release version and github stars by [@jdx](https://github.com/jdx) in [#604](#604) - add cross-site announcement banner by [@jdx](https://github.com/jdx) in [#600](#600) - fix banner dark-mode contrast by [@jdx](https://github.com/jdx) in [#601](#601) - respect banner expires field by [@jdx](https://github.com/jdx) in [#602](#602) - prefix star count with ★ glyph and populate it on deploy by [@jdx](https://github.com/jdx) in [#606](#606) - integrate Commander.js, oclif and yargs by [@gaojunran](https://github.com/gaojunran) in [#616](#616) - integrate Typer and Click by [@gaojunran](https://github.com/gaojunran) in [#619](#619) ### 🔍 Other Changes - **(docs)** remove shrill.en.dev analytics script by [@jdx](https://github.com/jdx) in [#614](#614) - **(release)** append en.dev sponsor blurb to release notes by [@jdx](https://github.com/jdx) in [#598](#598) - switch analytics from gtm to plausible by [@jdx](https://github.com/jdx) in [#609](#609) - pin taiki-e/install-action to commit SHA by [@jdx](https://github.com/jdx) in [#610](#610) - rename CLAUDE.md to AGENTS.md and symlink by [@jdx](https://github.com/jdx) in [#618](#618) ### 📦️ Dependency Updates - bump communique to 1.1.2 by [@jdx](https://github.com/jdx) in [#605](#605) - lock file maintenance by [@renovate[bot]](https://github.com/renovate[bot]) in [#607](#607) - update autofix-ci/action action to v1.3.4 by [@renovate[bot]](https://github.com/renovate[bot]) in [#611](#611) - update apple-actions/import-codesign-certs action to v7 by [@renovate[bot]](https://github.com/renovate[bot]) in [#612](#612) - update taiki-e/install-action digest to fc9eae0 by [@renovate[bot]](https://github.com/renovate[bot]) in [#613](#613) --------- Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
tmeijn
pushed a commit
to tmeijn/dotfiles
that referenced
this pull request
May 8, 2026
This MR contains the following updates: | Package | Update | Change | |---|---|---| | [usage](https://github.com/jdx/usage) | minor | `3.2.1` → `3.3.0` | MR created with the help of [el-capitano/tools/renovate-bot](https://gitlab.com/el-capitano/tools/renovate-bot). **Proposed changes to behavior should be submitted there as MRs.** --- ### Release Notes <details> <summary>jdx/usage (usage)</summary> ### [`v3.3.0`](https://github.com/jdx/usage/blob/HEAD/CHANGELOG.md#330---2026-05-03) [Compare Source](jdx/usage@v3.2.1...v3.3.0) ##### 🚀 Features - **(complete)** auto-completion for usage shebang scripts by [@​jdx](https://github.com/jdx) in [#​620](jdx/usage#620) ##### 🐛 Bug Fixes - **(docs)** stack banner and pin close button on mobile by [@​jdx](https://github.com/jdx) in [#​603](jdx/usage#603) ##### 📚 Documentation - **(site)** show release version and github stars by [@​jdx](https://github.com/jdx) in [#​604](jdx/usage#604) - add cross-site announcement banner by [@​jdx](https://github.com/jdx) in [#​600](jdx/usage#600) - fix banner dark-mode contrast by [@​jdx](https://github.com/jdx) in [#​601](jdx/usage#601) - respect banner expires field by [@​jdx](https://github.com/jdx) in [#​602](jdx/usage#602) - prefix star count with ★ glyph and populate it on deploy by [@​jdx](https://github.com/jdx) in [#​606](jdx/usage#606) - integrate Commander.js, oclif and yargs by [@​gaojunran](https://github.com/gaojunran) in [#​616](jdx/usage#616) - integrate Typer and Click by [@​gaojunran](https://github.com/gaojunran) in [#​619](jdx/usage#619) ##### 🔍 Other Changes - **(docs)** remove shrill.en.dev analytics script by [@​jdx](https://github.com/jdx) in [#​614](jdx/usage#614) - **(release)** append en.dev sponsor blurb to release notes by [@​jdx](https://github.com/jdx) in [#​598](jdx/usage#598) - switch analytics from gtm to plausible by [@​jdx](https://github.com/jdx) in [#​609](jdx/usage#609) - pin taiki-e/install-action to commit SHA by [@​jdx](https://github.com/jdx) in [#​610](jdx/usage#610) - rename CLAUDE.md to AGENTS.md and symlink by [@​jdx](https://github.com/jdx) in [#​618](jdx/usage#618) ##### 📦️ Dependency Updates - bump communique to 1.1.2 by [@​jdx](https://github.com/jdx) in [#​605](jdx/usage#605) - lock file maintenance by [@​renovate\[bot\]](https://github.com/renovate\[bot]) in [#​607](jdx/usage#607) - update autofix-ci/action action to v1.3.4 by [@​renovate\[bot\]](https://github.com/renovate\[bot]) in [#​611](jdx/usage#611) - update apple-actions/import-codesign-certs action to v7 by [@​renovate\[bot\]](https://github.com/renovate\[bot]) in [#​612](jdx/usage#612) - update taiki-e/install-action digest to [`fc9eae0`](jdx/usage@fc9eae0) by [@​renovate\[bot\]](https://github.com/renovate\[bot]) in [#​613](jdx/usage#613) </details> --- ### Configuration 📅 **Schedule**: (UTC) - Branch creation - At any time (no schedule defined) - Automerge - At any time (no schedule defined) 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever MR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this MR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this MR, check this box --- This MR has been generated by [Mend Renovate](https://github.com/renovatebot/renovate). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0My4xNjguNSIsInVwZGF0ZWRJblZlciI6IjQzLjE2OC41IiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJSZW5vdmF0ZSBCb3QiLCJhdXRvbWF0aW9uOmJvdC1hdXRob3JlZCIsImRlcGVuZGVuY3ktdHlwZTo6bWlub3IiXX0=-->
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Removes the proxied shrill.en.dev analytics snippet from the VitePress docs config — the endpoint is going away.
Note
Low Risk
Low risk documentation-site change; it only removes a third-party analytics script include and should not affect core functionality beyond stopping analytics events.
Overview
Removes the
shrill.en.devanalytics<script>(and itsdata-domain/data-apiattributes) from the VitePressheadconfig for the docs site, leaving only the existing OpenGraph/Twitter meta tags.Reviewed by Cursor Bugbot for commit 52c5f7e. Bugbot is set up for automated code reviews on this repo. Configure here.