Skip to content

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

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

chore(docs): remove shrill.en.dev analytics script#903
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: only removes a client-side analytics snippet from docs; no functional or data-path logic changes beyond tracking.

Overview
Removes the shrill.en.dev analytics <script> entry from docs/.vitepress/config.mts head, so the docs site no longer loads the proxied tracking script.

No other docs configuration or rendering behavior is changed.

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

@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented May 2, 2026

Greptile Summary

Removes the proxied shrill.en.dev analytics <script> tag from the VitePress head config in docs/.vitepress/config.mts. The change is a clean, one-hunk deletion with no other references to the endpoint remaining in the file.

Confidence Score: 5/5

Safe to merge — single-line documentation config cleanup with no logic changes.

The PR removes a single script element from a docs config file. No logic, no runtime behaviour, and no other references to the removed endpoint remain. No issues found.

No files require special attention.

Important Files Changed

Filename Overview
docs/.vitepress/config.mts Removes the shrill.en.dev analytics script tag from the VitePress head config; no remaining references to the endpoint.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["VitePress Build"] --> B["docs/.vitepress/config.mts"]
    B --> C["head[] entries"]
    C --> D["OpenGraph meta tags"]
    C --> E["Other head entries"]
    F["~~shrill.en.dev analytics script~~\n(removed)"] -. "was here" .-> C
    style F stroke-dasharray: 5 5,fill:#fee2e2,color:#991b1b
Loading

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

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 an external analytics script from the VitePress configuration file. I have no feedback to provide as there are no review comments to evaluate.

@jdx jdx merged commit ae7c6af into main May 2, 2026
23 checks passed
@jdx jdx deleted the chore/remove-shrill-analytics branch May 2, 2026 13:44
@jdx jdx mentioned this pull request May 2, 2026
jdx added a commit that referenced this pull request May 5, 2026
### 🚀 Features

- **(builtins)** add `buildifier` format and lint built-ins by
[@plx](https://github.com/plx) in
[#896](#896)

### 🐛 Bug Fixes

- **(step)** only auto-batch when rendered command exceeds ARG_MAX by
[@jdx](https://github.com/jdx) in
[#901](#901)

### 📚 Documentation

- thank Namespace for GitHub Actions runner support by
[@jdx](https://github.com/jdx) in
[#895](#895)

### 🔍 Other Changes

- **(ci)** use !cancelled() instead of always() for final job by
[@jdx](https://github.com/jdx) in
[#906](#906)
- **(docs)** remove shrill.en.dev analytics script by
[@jdx](https://github.com/jdx) in
[#903](#903)
- remove rust-cache from release jobs by [@jdx](https://github.com/jdx)
in [#893](#893)
- invert CLAUDE.md/AGENTS.md so AGENTS.md is canonical by
[@jdx](https://github.com/jdx) in
[#905](#905)
- set dev profile debug to 1 by [@jdx](https://github.com/jdx) in
[#907](#907)

### 📦️ Dependency Updates

- update anthropics/claude-code-action digest to fefa07e by
[@renovate[bot]](https://github.com/renovate[bot]) in
[#897](#897)
- update jdx/mise-action digest to 1648a78 by
[@renovate[bot]](https://github.com/renovate[bot]) in
[#898](#898)
- update apple-actions/import-codesign-certs action to v7 by
[@renovate[bot]](https://github.com/renovate[bot]) in
[#900](#900)
- update autofix-ci/action action to v1.3.4 by
[@renovate[bot]](https://github.com/renovate[bot]) in
[#899](#899)
- lock file maintenance by
[@renovate[bot]](https://github.com/renovate[bot]) in
[#908](#908)

### New Contributors

- @plx made their first contribution in
[#896](#896)

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> **Low Risk**
> Low risk release bookkeeping: primarily version string updates across
manifests and docs with no functional code changes in this diff.
> 
> **Overview**
> Updates the project for the `v1.45.0` release by bumping the crate/CLI
version (`Cargo.toml`, `Cargo.lock`, `hk.usage.kdl`, generated CLI docs)
and adding the `1.45.0` entry to `CHANGELOG.md`.
> 
> Refreshes documentation and example configs to reference the new
versioned Pkl package URLs (`docs/*.md`, `docs/public/*.pkl`,
`hk-example.pkl`).
> 
> <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit
cfe2da5. Bugbot is set up for automated
code reviews on this repo. Configure
[here](https://www.cursor.com/dashboard/bugbot).</sup>
<!-- /CURSOR_SUMMARY -->

Co-authored-by: mise-en-dev <[email protected]>
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