Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Remove/olduserdoc#279

Merged
gihanmudalige merged 2 commits into
developfrom
remove/olduserdoc
Mar 9, 2026
Merged

Remove/olduserdoc#279
gihanmudalige merged 2 commits into
developfrom
remove/olduserdoc

Conversation

@gihanmudalige

Copy link
Copy Markdown
Collaborator

Remove Old user.tex which is superseded by readthedocs documentation.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Removes legacy LaTeX-based user documentation and related build artifacts in favor of the existing ReadTheDocs/Sphinx documentation flow.

Changes:

  • Delete doc/user.tex and the old doc/build_docs.sh script.
  • Wire the “Key Concepts” content into the Sphinx/ReadTheDocs docs (index.rst) and refresh cross-references.
  • Simplify doc/Makefile to remove user.pdf/minted-related cleanup paths that are no longer relevant.

Reviewed changes

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

Show a summary per file
File Description
doc/user.tex Removes superseded LaTeX user manual.
doc/build_docs.sh Removes obsolete docs build helper script.
doc/Makefile Drops user-specific targets/cleanup to match removal of LaTeX manual.
doc/keyconcept.md Updates title casing and replaces LaTeX-style section reference with an explicit link into existing docs.
doc/index.rst Adds keyconcept.md to the Sphinx toctree so it is published on ReadTheDocs.

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

You can also share your feedback on Copilot code review. Take the survey.

@gihanmudalige gihanmudalige merged commit 0ab856c into develop Mar 9, 2026
4 of 5 checks passed
@gihanmudalige gihanmudalige deleted the remove/olduserdoc branch March 9, 2026 18:14
gihanmudalige added a commit that referenced this pull request Apr 14, 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