Skip to content

docs: update WorldCup2026 README to remove stale sections#68

Merged
TimWheeler-SQUP merged 2 commits into
mainfrom
work/tw/worldcup2026-readme-update
Jun 17, 2026
Merged

docs: update WorldCup2026 README to remove stale sections#68
TimWheeler-SQUP merged 2 commits into
mainfrom
work/tw/worldcup2026-readme-update

Conversation

@TimWheeler-SQUP

Copy link
Copy Markdown
Contributor

Updates ReadMe to be clearer and remove incorrect information.

Removes the 'What gets indexed', 'Configuration', and 'Known limitations'
sections and corrects the data source URL reference.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@TimWheeler-SQUP TimWheeler-SQUP added the documentation Improvements or additions to documentation label Jun 17, 2026
@TimWheeler-SQUP TimWheeler-SQUP requested a review from a team June 17, 2026 11:40

@clarkd clarkd 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.

Claude might flag a version bump here, but not needed given it's just README changes

@clarkd

clarkd commented Jun 17, 2026

Copy link
Copy Markdown
Member

@claude review once

Comment thread plugins/WorldCup2026/v1/docs/README.md
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown

🧩 Plugin PR Summary

📦 Modified Plugins

  • plugins/WorldCup2026/v1

📋 Results

Step Status
Validation ✅ Passed
Deployment 🚀 Deployed

🔍 Validation Details

worldcup2026
{
  "valid": true,
  "pluginName": "worldcup2026",
  "pluginType": "hybrid",
  "summary": {
    "Data Streams": 7,
    "Import Definitions": 1,
    "UI Configuration": false,
    "Has Icon": true,
    "Has Default Content": true,
    "Config Validation": true,
    "Custom Types": true
  }
}

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Version bump added as requested — docs-only change is good to merge.

Extended reasoning...

Overview

This PR removes three stale sections ("What gets indexed", "Configuration", "Known limitations") from the WorldCup2026 plugin README and bumps the plugin version from 1.2.0 to 1.2.1. Two files changed: plugins/WorldCup2026/v1/docs/README.md and plugins/WorldCup2026/v1/metadata.json.

Security risks

None. This is a documentation-only change plus a version string update. No code, credentials, network behavior, or auth surfaces are touched.

Level of scrutiny

Minimal. Docs-only deletion of stale content with the required patch version bump per repo policy. The plugin uses authMode: none and has no UI configuration, so the removed "Configuration" section was indeed accurate to remove.

Other factors

My prior review on this PR flagged the missing version bump; the author has addressed it (1.2.0 → 1.2.1, appropriate patch bump for a docs change). No outstanding reviewer comments remain. The bug hunting system found no issues.

@TimWheeler-SQUP TimWheeler-SQUP requested a review from clarkd June 17, 2026 15:55
@TimWheeler-SQUP TimWheeler-SQUP merged commit 946fe43 into main Jun 17, 2026
2 checks passed
@TimWheeler-SQUP TimWheeler-SQUP deleted the work/tw/worldcup2026-readme-update branch June 17, 2026 17:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Development

Successfully merging this pull request may close these issues.

2 participants