Skip to content

Add Plausible analytics - #38

Merged
micokoch merged 1 commit into
mainfrom
micokoch-plausible
Aug 18, 2026
Merged

micokoch merged 1 commit into
mainfrom
micokoch-plausible

Conversation

@micokoch

Copy link
Copy Markdown
Member

Adds privacy-friendly Plausible analytics by injecting the shared reichlab.io tracking snippet into <head>. Part of consolidating all reichlab.io sites under a single Plausible site.

🤖 Generated with Claude Code

Injects the shared reichlab.io Plausible snippet so this site is tracked
in the consolidated reichlab.io Plausible site.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@micokoch micokoch self-assigned this Aug 14, 2026
@micokoch micokoch added the enhancement New feature or request label Aug 14, 2026

@micokoch micokoch left a comment •

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This PR adds the shared reichlab.io Plausible snippet to the pkgdown site via template.includes.in_header in _pkgdown.yml (feeding the consolidated reichlab.io Plausible site, pa-DOosb…).

Testing: _pkgdown.yml validated as parseable YAML. This uses the identical template.includes.in_header mechanism as the hubverse pkgdown sites where I confirmed it renders into <head> (e.g. hubValidations, hub-data), so I relied on config validation + that proven mechanism rather than spinning up a full local pkgdown/R build here.

On the failing lint check: it's pre-existing and unrelated to this PR. The only file changed is _pkgdown.yml, and lintr::lint_package() lints R source and vignettes — all of its warnings are in existing files (R/ext.R, R/int.R, R/make_dist_fns.R, R/utils.R, vignettes/distfromq.Rmd) that this PR doesn't touch, so it fails identically on main. The lint cleanup (implicit returns, |> vs %>%) is separate maintenance.

@matthewcornell matthewcornell left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Approved, but noting failing lint checks.

@micokoch
micokoch merged commit a9cf20d into main Aug 18, 2026
6 of 7 checks passed
@micokoch
micokoch deleted the micokoch-plausible branch August 18, 2026 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants