Skip to content

Update Matomo script comments and formatting#502

Open
tjeerdijk wants to merge 1 commit intomasterfrom
tjeerdijk-patch-1
Open

Update Matomo script comments and formatting#502
tjeerdijk wants to merge 1 commit intomasterfrom
tjeerdijk-patch-1

Conversation

@tjeerdijk
Copy link
Contributor

@tjeerdijk tjeerdijk commented Feb 6, 2026

removed enclosing <script> tags

Summary by CodeRabbit

  • Chores
    • Simplified Matomo analytics script configuration in documentation static assets.

removed enclosing <script> tags
@tjeerdijk tjeerdijk requested a review from matteopilz February 6, 2026 16:02
@coderabbitai
Copy link

coderabbitai bot commented Feb 6, 2026

📝 Walkthrough

Walkthrough

A Matomo analytics script snippet in the documentation's static files was modified by removing HTML script wrapper tags and slightly rewording the code comment. The underlying analytics initialization logic remains unchanged.

Changes

Cohort / File(s) Summary
Matomo Analytics Script
docs/source/_static/matomo.js
Removed <script> and </script> tags from the snippet and updated the comment phrasing from "script below is used by" to "script is used by". All functional code remains intact.

Poem

🐰 A script once wrapped in HTML's embrace,
Now stands as pure JavaScript in place,
The tags have fled, the comment refined,
Analytics flow with a cleaner mind!
Matomo's watch continues, sleek and free,
Our analytics more elegant to see! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive The title mentions 'comments and formatting' but the main change is removing script tags, which is structural modification, not just formatting. Consider a more specific title like 'Remove script tags from Matomo snippet' to better reflect the primary structural change.
✅ 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.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch tjeerdijk-patch-1

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.

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