-
Notifications
You must be signed in to change notification settings - Fork 125
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
Idealised moist docs #187
Idealised moist docs #187
Conversation
are we supposed to add an Authors section or is that optional? |
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.
Really really nice!
I think the only issue I can find is that the file itself (and therefore in the index) has idealized spelt with an s (correctly) not a z (as per the model code), but I think we can ignore this!
Thanks for checking it, I've added the authors at the bottom, I forgot about that. Happy to merge? |
Merge time! |
yes! :) |
Here are the corrections to my previous pull request.
Apologies for the new branch, I decided to rename my local branch and this seems to have caused some problems with the remotes.