Skip to content

Add scarf#83

Merged
nscuro merged 1 commit into
mainfrom
scarf
May 18, 2026
Merged

Add scarf#83
nscuro merged 1 commit into
mainfrom
scarf

Conversation

@nscuro
Copy link
Copy Markdown
Member

@nscuro nscuro commented May 18, 2026

No description provided.

Signed-off-by: nscuro <nscuro@protonmail.com>
Copilot AI review requested due to automatic review settings May 18, 2026 16:27
@nscuro nscuro added the enhancement New feature or request label May 18, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a Scarf analytics tracking pixel to the docs site, enabled only on deployment builds via an environment variable, and adds a YouTube social link.

Changes:

  • New overrides/main.html MkDocs Material template override that injects a Scarf tracking pixel into <head> when extra.scarf_pixel is truthy.
  • mkdocs.yml wires up custom_dir: overrides, sets scarf_pixel from SCARF_ENABLED env var (defaulting false), and adds a YouTube social icon link.
  • .github/workflows/docs.yml sets SCARF_ENABLED=true for the deploy job so the pixel only loads on published builds.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
overrides/main.html New template override that conditionally injects the Scarf pixel <img> in extrahead.
mkdocs.yml Enables custom theme overrides, declares scarf_pixel extra from env, adds YouTube social link.
.github/workflows/docs.yml Sets SCARF_ENABLED: "true" env var for the docs deploy job.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@nscuro nscuro merged commit e59ba5c into main May 18, 2026
6 checks passed
@nscuro nscuro deleted the scarf branch May 18, 2026 16:31
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