Skip to content

Conversation

jstirnaman
Copy link
Collaborator

@jstirnaman jstirnaman commented Aug 21, 2025

…ments

  • Add GitHub Actions workflow to detect README changes and create sync reminders
  • Update README_TEMPLATE.md with documentation sync instructions
  • Add validate_readme.py script for template compliance checking
  • Update CONTRIBUTING.md with comprehensive documentation sync process guide

The workflow automatically:

  • Detects plugin README changes on master branch commits
  • Creates commit comments with pre-filled sync request links to docs-v2
  • Provides clear instructions for the sync process
  • Requires no secrets or cross-repository authentication

Updated documentation includes:

  • Complete sync workflow instructions
  • Template compliance requirements
  • Automated sync process explanation
  • Manual sync alternatives

This workflow works with influxdata/docs-v2#6329 to sync updated READMEs in influxdb3_plugins to influxdata/docs-v2.

…ments

- Add GitHub Actions workflow to detect README changes and create sync reminders
- Update README_TEMPLATE.md with documentation sync instructions
- Add validate_readme.py script for template compliance checking
- Update CONTRIBUTING.md with comprehensive documentation sync process guide

The workflow automatically:
- Detects plugin README changes on master branch commits
- Creates commit comments with pre-filled sync request links to docs-v2
- Provides clear instructions for the sync process
- Requires no secrets or cross-repository authentication

Updated documentation includes:
- Complete sync workflow instructions
- Template compliance requirements
- Automated sync process explanation
- Manual sync alternatives
Copilot

This comment was marked as outdated.

Copilot

This comment was marked as outdated.

…ers, move to scripts directory:- Moves validate_readme.py to /scripts

- Replaces magic numbers with variables
- Adds support for --help and --list flags- Updates contributing doc
@jstirnaman jstirnaman force-pushed the jstirnaman/validate-and-sync-readmes-with-influxdata-docs branch from 3bc228e to cf7a557 Compare August 21, 2025 21:43
@jstirnaman jstirnaman requested a review from Copilot August 21, 2025 21:43
Copy link
Contributor

@Copilot 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

This PR adds an automated documentation sync system to remind contributors to synchronize plugin README changes with the InfluxDB documentation site. The workflow automatically detects when plugin READMEs are modified and provides clear instructions for syncing those changes to the docs-v2 repository.

Key changes:

  • Automated detection and reminder system for README changes via GitHub Actions
  • Comprehensive README template with sync instructions and validation requirements
  • Python validation script to ensure README compliance with template standards

Reviewed Changes

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

File Description
.github/workflows/remind-sync-docs.yml GitHub Actions workflow that detects README changes and creates sync reminder comments
README_TEMPLATE.md Comprehensive template for plugin READMEs with sync instructions and proper structure
CONTRIBUTING.md Updated with detailed documentation sync process and requirements
scripts/validate_readme.py Python script for validating README compliance against template standards
Comments suppressed due to low confidence (1)

@MeelahMe MeelahMe requested a review from Copilot October 9, 2025 19:12
Copy link
Contributor

@Copilot 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

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

@MeelahMe
Copy link
Collaborator

MeelahMe commented Oct 9, 2025

So far I have:

  • Changed workflow now triggers on 'main' branch instead of 'master' that appeared to be causing an error
  • Updated .github/workflows/remind-sync-docs.yml
  • Improved validation script flexibility
  • Fixed basic_transformation README structure
  • Tested locally and confirmed the workflow detection logic works

@MeelahMe MeelahMe requested a review from Copilot October 11, 2025 00:33
Copy link
Contributor

@Copilot 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

Copilot reviewed 5 out of 5 changed files in this pull request and generated 4 comments.

@MeelahMe
Copy link
Collaborator

MeelahMe commented Oct 11, 2025

@jstirnaman

Updates Summary

Here's what I did:

Main Fix

  • Fixed the workflow to trigger on main branch instead of master - this looked to be the reason for the error. Workflow was not triggering in master.

Improvements

  • Cleaned up the validation script - reduced false positive errors from 7+ down to 0. The script now accurately validates plugin READMEs.
  • Addressed Copilot feedback - removed the 10-file limit (so we won't miss changes if lots of plugins are updated) and moved some hardcoded values to constants for easier maintenance.

Testing

Tested locally:

  • Workflow detection logic correctly identifies changed READMEs
  • Validation script passes with 0 errors on test plugins
  • All code quality checks passing

Status: Ready for review

@MeelahMe MeelahMe marked this pull request as ready for review October 11, 2025 01:01
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.

3 participants