Skip to content

Review: remove SEO tools setting#415

Draft
Copilot wants to merge 1 commit intoremove/seo_toolsfrom
copilot/sub-pr-414
Draft

Review: remove SEO tools setting#415
Copilot wants to merge 1 commit intoremove/seo_toolsfrom
copilot/sub-pr-414

Conversation

Copy link

Copilot AI commented Mar 4, 2026

PR #414 removes two previously added analytics/tracking integrations from the Jekyll site.

Changes reviewed

  • _config.yml: analytics.provider set to false, disabling Google Analytics
  • _includes/analytics-providers/azure-application-insights.html: Deleted — contained hardcoded InstrumentationKey connection string; removal is a security improvement
  • _includes/seo.html: Removed {% include %} reference to the now-deleted Application Insights file
  • _includes/analytics-providers/google.html: Reverted from hardcoded gtag.js (G-GYD44HY4T3) to the legacy _gaq template variable form — functionally inert since analytics.provider is false, but worth noting it introduces deprecated Universal Analytics code (discontinued 2023) rather than leaving the file as-is or clearing it

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title [WIP] Remove SEO tools setting Review: remove SEO tools setting Mar 4, 2026
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.

2 participants