Skip to content

Many fixes to the documentation#389

Merged
xylar merged 1 commit intoMPAS-Dev:masterfrom
xylar:fix_docstrings_and_docs
May 13, 2022
Merged

Many fixes to the documentation#389
xylar merged 1 commit intoMPAS-Dev:masterfrom
xylar:fix_docstrings_and_docs

Conversation

@xylar
Copy link
Copy Markdown
Collaborator

@xylar xylar commented May 13, 2022

Some API fixes, some title fixes, some formatting of code blocks.

Some API fixes, some title fixes, some formatting of code blocks.
@xylar xylar added bug Something isn't working dependencies and deployment Changes relate to creating conda and Spack environments, and creating a load script labels May 13, 2022
@xylar xylar requested a review from trhille May 13, 2022 20:47
@xylar xylar self-assigned this May 13, 2022
@xylar
Copy link
Copy Markdown
Collaborator Author

xylar commented May 13, 2022

@trhille, in building the docs for #348, I noticed these small issues. Just flagging you to make sure the fixes are correct. Make sure to build the docs as part if your testing. (I forget to test this sometime, too.)

@xylar xylar removed the request for review from trhille May 13, 2022 21:44
@xylar
Copy link
Copy Markdown
Collaborator Author

xylar commented May 13, 2022

I'm going to go ahead and merge. With these fixes, the docs built locally without any errors.

@xylar xylar merged commit c8f1b60 into MPAS-Dev:master May 13, 2022
@xylar xylar deleted the fix_docstrings_and_docs branch May 13, 2022 21:45
@trhille
Copy link
Copy Markdown
Collaborator

trhille commented May 16, 2022

@xylar, thanks for catching this, and apologies for the oversight.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working dependencies and deployment Changes relate to creating conda and Spack environments, and creating a load script

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants