Skip to content

Dev notes#133

Merged
ocefpaf merged 2 commits intoTEOS-10:mainfrom
ocefpaf:dev_notes
Apr 18, 2024
Merged

Dev notes#133
ocefpaf merged 2 commits intoTEOS-10:mainfrom
ocefpaf:dev_notes

Conversation

@ocefpaf
Copy link
Member

@ocefpaf ocefpaf commented Apr 10, 2023

Closes #129

  • Add a note about the matlab code for docstring parsing
  • Add a note about the C code for func parsing

@ocefpaf ocefpaf marked this pull request as ready for review April 10, 2023 13:04
@ocefpaf ocefpaf marked this pull request as draft April 10, 2023 14:03
@ocefpaf ocefpaf marked this pull request as ready for review April 10, 2023 14:26
@ocefpaf ocefpaf requested a review from efiring April 10, 2023 14:26
@ocefpaf
Copy link
Member Author

ocefpaf commented Apr 18, 2024

@efiring I want to incorporate this into our docs and test the process myself but I'll leave that to another PR. Do you mind taking a quick look if you have some free time so we can close this one?

@efiring
Copy link
Member

efiring commented Apr 18, 2024

Thank you. Sorry to have forgotten about this, long ago. I will try to review it in the next few days.

Copy link
Member

@efiring efiring left a comment

Choose a reason for hiding this comment

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

Looks fine.

@efiring
Copy link
Member

efiring commented Apr 18, 2024

I will leave it to you to merge when you are ready.

@ocefpaf
Copy link
Member Author

ocefpaf commented Apr 18, 2024

Thanks! I'll tackle the HTML docs in another PR. Let's get this one in so others have some breadcrumbs to follow when contributing to this repo.

@ocefpaf ocefpaf merged commit bf72545 into TEOS-10:main Apr 18, 2024
@ocefpaf ocefpaf deleted the dev_notes branch April 18, 2024 18:54
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.

Add a "Development Instructions" README/Docs

2 participants