-
Notifications
You must be signed in to change notification settings - Fork 0
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
Unit test #20
Unit test #20
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. I'll go through and add some commits today, then let me know if you're alright with the changes
# API | ||
|
||
This page contains documentation of the public API of MRINavigator. In the Julia REPL one can access this documentation by entering the help mode with ? and then writing the function for which the documentation should be shown. | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add the actual command in code: ? <my-function-name>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed
@Laura2305 I've done my review of this branch, if it goes through the CI with my changes, please review them and let me know when I can approve and merge |
@alexjaffray I have reviewed your changes and everything looks good to me. I'd first merge the documentation branch into setup (pull request #21) and then everything into main. |
@Laura2305 this should pass CI. Once it does, I will merge into main |
No description provided.