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

Updates to the readthedocs #266

Merged
merged 36 commits into from
Aug 26, 2024
Merged

Updates to the readthedocs #266

merged 36 commits into from
Aug 26, 2024

Conversation

blychs
Copy link
Collaborator

@blychs blychs commented Jul 31, 2024

Dear all,
This are the updates to the readthedocs that I mentioned on Monday. @rrbuchholz has already given them a quick look and agrees that I should submit the PR.
Please take into account that the names of some sections have changed, which we hope will make it more clear.
You will see a list of commits, all of them making sure that they only involve one real change.
If there is any change you disagree with, I am happy to roll that back.
Cheers,
Pablo

@blychs
Copy link
Collaborator Author

blychs commented Aug 9, 2024

@rschwant @zmoon

docs/develop/information_for_devs.rst Outdated Show resolved Hide resolved
docs/appendix/troubleshooting.rst Outdated Show resolved Hide resolved
docs/appendix/troubleshooting.rst Outdated Show resolved Hide resolved
docs/appendix/troubleshooting.rst Outdated Show resolved Hide resolved
docs/appendix/machine-specific-install.rst Show resolved Hide resolved
docs/appendix/troubleshooting.rst Outdated Show resolved Hide resolved
docs/appendix/troubleshooting.rst Outdated Show resolved Hide resolved
	Mention possible problems with Cartopy.crs
docs/appendix/troubleshooting.rst Outdated Show resolved Hide resolved
docs/appendix/troubleshooting.rst Outdated Show resolved Hide resolved
docs/appendix/troubleshooting.rst Outdated Show resolved Hide resolved
docs/getting_started/installation.rst Outdated Show resolved Hide resolved
docs/getting_started/installation.rst Outdated Show resolved Hide resolved
docs/getting_started/installation.rst Outdated Show resolved Hide resolved
@blychs
Copy link
Collaborator Author

blychs commented Aug 26, 2024

@zmoon hopefully this does it now.

@zmoon
Copy link
Collaborator

zmoon commented Aug 26, 2024

The linkcheck error (getting_started/installation: line 63) broken https://github.com/NOAA-CSL/MELODIES-MONET/tree/develop/python_env_ymls/casper is expected (dir renamed).

@blychs
Copy link
Collaborator Author

blychs commented Aug 26, 2024

The linkcheck error (getting_started/installation: line 63) broken https://github.com/NOAA-CSL/MELODIES-MONET/tree/develop/python_env_ymls/casper is expected (dir renamed).

Do I need to change anything to make this work?

@zmoon
Copy link
Collaborator

zmoon commented Aug 26, 2024

Do I need to change anything to make this work?

Nope, I was just noting, for posterity, that although it did error, it was an expected error. Right now it fails, since python_env_yamls/casper doesn't exist in the MM develop branch, but it should pass when this is merged.

Copy link
Collaborator

@zmoon zmoon left a comment

Choose a reason for hiding this comment

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

Thanks @blychs !

@zmoon zmoon merged commit 3121fc4 into NOAA-CSL:develop Aug 26, 2024
3 of 4 checks passed
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