Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

📝Moved function definitions for _all functions in motors to their own … #606

Conversation

aberiggs
Copy link
Contributor

@aberiggs aberiggs commented Oct 3, 2023

…category

Summary:

Moved function definitions for _all functions to the bottom of the file into their own category in doxygen

Also made a couple of typo corrections

Motivation:

Clean up documentation and move *_all function definitions to their own category since they're less commonly used on standalone motors.

References (optional):

Test Plan:

N/A

@aberiggs aberiggs requested a review from noam987 October 3, 2023 02:37
@noam987 noam987 changed the title Moved function definitions for _all functions in motors to their own … 📝Moved function definitions for _all functions in motors to their own … Oct 5, 2023
Copy link
Contributor

@noam987 noam987 left a comment

Choose a reason for hiding this comment

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

LGTM

@noam987 noam987 merged commit 696be0f into develop-pros-4 Oct 5, 2023
1 check passed
@noam987 noam987 deleted the pros-4/bugfix/move-definitions-for-_all-motor-functions branch October 5, 2023 14:08
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