Skip to content

docs: upgrade README badges and add auto-sync to introduction.rst#250

Merged
yuecideng merged 1 commit into
mainfrom
enhance/readme-badges-and-docs-sync
Apr 29, 2026
Merged

docs: upgrade README badges and add auto-sync to introduction.rst#250
yuecideng merged 1 commit into
mainfrom
enhance/readme-badges-and-docs-sync

Conversation

@yuecideng
Copy link
Copy Markdown
Contributor

Description

This PR upgrades the README badge style and adds automated synchronization between README.md and docs/source/introduction.rst.

Badge upgrades:

  • All existing badges upgraded to for-the-badge style (taller, bolder, more modern look)
  • Added Python logo to the Python version badge

Auto-sync mechanism:

  • New script docs/scripts/sync_readme.py converts README.md → introduction.rst using pypandoc
  • Post-processes pandoc output to fix Sphinx-specific formatting (badges, NOTE directives, image paths, code blocks)
  • Hooked into docs/Makefile so docs always build from the latest README.md
  • Added sync step to CI workflow (.github/workflows/main.yml)
  • Supports --check mode for CI staleness validation

Other updates:

  • Updated roadmap content to reflect current priorities

Dependencies: pypandoc_binary added to docs/requirements.txt

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • Enhancement (non-breaking change which improves an existing functionality)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (existing functionality will not work without user modification)
  • Documentation update

Screenshots

Badges before (default style) → after (for-the-badge style):

Before:
before

After:
after

Checklist

  • I have run the black . command to format the code base.
  • I have made corresponding changes to the documentation
  • Dependencies have been updated, if applicable.

🤖 Generated with Claude Code

Upgrade README badge style to for-the-badge for a bolder, more modern
look and add pypandoc-based sync script that auto-generates
docs/source/introduction.rst from README.md, hooked into both the
Makefile and CI workflow. Also update roadmap content.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings April 29, 2026 11:18
@yuecideng yuecideng added docs Improvements or additions to documentation enhancement New feature or request labels Apr 29, 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

This PR modernizes the project’s README badge styling and introduces an automated mechanism to keep the Sphinx docs introduction page synchronized with README.md, so the docs site reflects the latest project overview.

Changes:

  • Updated README badges to for-the-badge style (including Python logo on the Python badge).
  • Added docs/scripts/sync_readme.py to convert README.mddocs/source/introduction.rst via pandoc + post-processing.
  • Wired the sync step into docs builds (Makefile) and CI, and refreshed roadmap wording.

Reviewed changes

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

Show a summary per file
File Description
README.md Updates shields badges and tweaks intro wording to match current positioning.
docs/scripts/sync_readme.py New README→RST conversion + Sphinx-specific cleanup logic.
docs/source/introduction.rst Updated to match README content and new generation format.
docs/Makefile Runs README→introduction sync automatically before Sphinx build targets.
docs/requirements.txt Adds pandoc binary dependency for conversion.
.github/workflows/main.yml Runs the sync step during docs build in CI.
docs/source/resources/roadmap.md Roadmap copy updates reflecting current priorities.

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

Comment on lines +57 to +58
`CONTRIBUTING.md <CONTRIBUTING.md>`__ file in this repository for
guidelines on how to get started.
Copy link

Copilot AI Apr 29, 2026

Choose a reason for hiding this comment

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

This hyperlink is currently relative (CONTRIBUTING.md) and will point to a non-existent page in the built documentation output (likely a 404). Use an absolute GitHub URL, or add CONTRIBUTING to the Sphinx source tree and link with :doc:/toctree so it resolves in the docs site.

Suggested change
`CONTRIBUTING.md <CONTRIBUTING.md>`__ file in this repository for
guidelines on how to get started.
`CONTRIBUTING.md <https://github.com/DexForce/EmbodiChain/blob/main/CONTRIBUTING.md>`__
file in this repository for guidelines on how to get started.

Copilot uses AI. Check for mistakes.
Comment on lines +112 to +116
if re.match(r"^\s+\[!NOTE\]", line):
note_match = re.match(r"^\s+\[!NOTE\]\s*(.*)", line)
note_text = note_match.group(1) if note_match else ""
note_text = note_text.replace("\\*", "*")
note_lines: list[str] = []
Copy link

Copilot AI Apr 29, 2026

Choose a reason for hiding this comment

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

The [!NOTE] conversion logic is producing malformed RST in the generated introduction.rst (the list markers end up inline with the preceding sentence). Update the NOTE post-processing to preserve list structure inside the directive (e.g., split wrapped/inlined * items onto their own lines and insert the required blank line before the list).

Copilot uses AI. Check for mistakes.
Comment thread .github/workflows/main.yml
Comment on lines +14 to +19
EmbodiChain is in Alpha and under active development: * More
features will be continually added in the coming months. You can find
more details in the
`roadmap <https://dexforce.github.io/EmbodiChain/resources/roadmap.html>`__.
* Since this is an early release, we welcome feedback (bug reports,
feature requests, etc.) via GitHub Issues.
Copy link

Copilot AI Apr 29, 2026

Choose a reason for hiding this comment

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

The generated NOTE admonition formatting is invalid reStructuredText: the bullet list items are being merged into the preceding sentence ("...development: * More ..."), which will render incorrectly and can trigger Sphinx warnings/errors (docs build uses -W). Regenerate this block so list items are on their own indented lines inside the directive (and typically separated from the introductory sentence by a blank line).

Suggested change
EmbodiChain is in Alpha and under active development: * More
features will be continually added in the coming months. You can find
more details in the
`roadmap <https://dexforce.github.io/EmbodiChain/resources/roadmap.html>`__.
* Since this is an early release, we welcome feedback (bug reports,
feature requests, etc.) via GitHub Issues.
EmbodiChain is in Alpha and under active development. You can find
more details in the
`roadmap <https://dexforce.github.io/EmbodiChain/resources/roadmap.html>`__.
* More features will be continually added in the coming months.
* Since this is an early release, we welcome feedback (bug reports,
feature requests, etc.) via GitHub Issues.

Copilot uses AI. Check for mistakes.
@yuecideng yuecideng merged commit c824c86 into main Apr 29, 2026
11 of 12 checks passed
@yuecideng yuecideng deleted the enhance/readme-badges-and-docs-sync branch April 29, 2026 17:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Improvements or additions to documentation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants