Skip to content

[PR] Sphinx documentation v.beta1.0#17

Merged
LycoKodo merged 2 commits intomainfrom
sphinx-documentation-v.beta1.0
Jul 28, 2025
Merged

[PR] Sphinx documentation v.beta1.0#17
LycoKodo merged 2 commits intomainfrom
sphinx-documentation-v.beta1.0

Conversation

@venus-beetroot
Copy link
Copy Markdown
Collaborator


name: PR/Fix Template
about: Create documentation startup for GravityLib in Readthedocs
title: "[PR] Sphinx documentation v.beta1.0"
labels: pullrequest
assignees: 'venus-beetroot'


Pull Request

Thank you for contributing to GravLib! Please complete the checklist and fill out the relevant sections.


Summary

Provide a short summary of what this PR changes, fixes, or adds.

This adds documentation in the form of .md files converted to .html through the myst-parser extension for readthedocs.io. I will add more documentation, formatting and files in the upcoming weeks as the project expands


Motivation

Why are you making this pull request?

We are creating documentation for this project as we expect it to grow and be a mainstream codebase for all V5, written in rust. Documentation allows us to be both organised, as well as allowing the public to read and understand what we did, what was added, and how to implement our code into their own robots.


Checklist Before Submission

  • My PR addresses a SINGLE concern or feature.
  • I HAVE TESTED the code and it compiles without warnings.
  • I HAVE NOT broken existing public APIs.
  • I HAVE documented new public APIs using Rustdoc comments.
  • I HAVE NOT introduced compiler warnings.
  • I HAVE checked that this PR passes existing unit tests.
    [N/A] I HAVE added unit tests for any new functionality or bug fixes (if applicable).
  • I HAVE not broken the continuous integration build.
  • I HAVE avoided major architectural changes unless previously discussed.
  • I HAVE adhered to GravLib naming conventions.
    [N/A] I HAVE tested on actual V5 hardware (if applicable).
  • I HAVE verified memory usage is within V5 constraints.
  • I HAVE checked compatibility with target VEX V5 brain firmware version.
  • I HAVE verified real-time performance is not negatively impacted.

Testing

Describe how you tested/plan to test this PR (unit tests, real hardware, simulation, etc.). If tested on hardware, specify which V5 components were used.

Refer to https://gravlib.readthedocs.io and build the changes made to default branch (main) through my account.


Additional Notes

Add any additional information, such as future work, limitations, or things to review closely.

N/A

Moved documentation source files from docs/source to the root of docs, updated the Makefile to use the new source directory, and regenerated Sphinx build outputs. This change simplifies the documentation structure and ensures the build process uses the correct paths.

Hopefully this is fixed now 😞
@LycoKodo
Copy link
Copy Markdown
Contributor

Do you want me to review this now?

Also the host still seems to not be working.

But its chill :D

@LycoKodo LycoKodo self-requested a review July 28, 2025 02:12
Copy link
Copy Markdown
Contributor

@LycoKodo LycoKodo left a comment

Choose a reason for hiding this comment

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

gw

@LycoKodo LycoKodo merged commit b0e8dcd into main Jul 28, 2025
3 checks passed
@LycoKodo
Copy link
Copy Markdown
Contributor

reverted, start another pr to continue this issue

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