Skip to content

docs: move developer instructions to bottom of README#43

Merged
jmf-pobox merged 1 commit intomainfrom
copilot/reorder-readme-content
Apr 26, 2026
Merged

docs: move developer instructions to bottom of README#43
jmf-pobox merged 1 commit intomainfrom
copilot/reorder-readme-content

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 26, 2026

README mixed user-facing and developer content, with the "For Developers (git clone)" block embedded mid-page inside Installation.

Changes

  • Extracted "For Developers (git clone)" subsection (dev setup, dev commands, ethos agent team) from Installation and moved it to a top-level section after Documentation, immediately before Contributing/License/Credits
  • Fixed cross-reference in Examples: "see For Developers above" → "below"
  • Demoted #### headings inside the moved section to ### to match the new top-level context

New section order: Why → Installation → Quick Start → What You Can Write → Usage → Syntax Reference → Examples → LaTeX Options → IDE → Overleaf → Troubleshooting → Limitations → Status → Documentation → For Developers → Contributing → License → Credits


Note

Low Risk
Documentation-only reordering with a small anchor reference tweak; no runtime or build logic changes.

Overview
Moves the README’s developer-focused setup (clone instructions, make commands, and ethos agent-team notes) out of the Installation area into a new top-level ## For Developers (git clone) section placed near the end of the document.

Updates the Examples section cross-reference to point below (since the section moved) and adjusts heading levels within the moved block to fit its new top-level placement.

Reviewed by Cursor Bugbot for commit fc81e33. Bugbot is set up for automated code reviews on this repo. Configure here.

@jmf-pobox jmf-pobox marked this pull request as ready for review April 26, 2026 16:24
@jmf-pobox jmf-pobox self-requested a review as a code owner April 26, 2026 16:24
Copilot AI review requested due to automatic review settings April 26, 2026 16:24
Copy link
Copy Markdown
Owner

@jmf-pobox jmf-pobox left a comment

Choose a reason for hiding this comment

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

Looks good.

@jmf-pobox jmf-pobox merged commit c825959 into main Apr 26, 2026
5 checks passed
@jmf-pobox jmf-pobox deleted the copilot/reorder-readme-content branch April 26, 2026 16:25
Copy link
Copy Markdown

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

Reorganizes the README to separate developer/contributor setup guidance from user-facing installation and usage docs, improving readability for end users.

Changes:

  • Moved the “For Developers (git clone)” block out of the Installation section into its own top-level section near the bottom of the README.
  • Updated the Examples cross-reference from “above” to “below”.
  • Adjusted heading levels within the moved developer section to match the new nesting.

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

Comment thread README.md
cd examples && make

# Convert a file
txt2tex myfile.txt
Copy link

Copilot AI Apr 26, 2026

Choose a reason for hiding this comment

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

The dev setup uses uv sync and earlier examples use uv run ..., but this section later suggests running txt2tex myfile.txt directly. Unless the user has activated the created virtualenv or installed txt2tex globally, txt2tex may not be on PATH. Consider switching this to uv run txt2tex ... (or explicitly mention activating .venv) to keep the workflow consistent and avoid “command not found” for contributors.

Suggested change
txt2tex myfile.txt
uv run txt2tex myfile.txt

Copilot uses AI. Check for mistakes.
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