Skip to content

Contributing code back to MOM6 Part 2 notes#26

Merged
chrisb13 merged 4 commits into
mainfrom
Part2-code-back2MOM6
Apr 30, 2026
Merged

Contributing code back to MOM6 Part 2 notes#26
chrisb13 merged 4 commits into
mainfrom
Part2-code-back2MOM6

Conversation

@chrisb13
Copy link
Copy Markdown
Collaborator

No description provided.

@chrisb13 chrisb13 marked this pull request as ready for review April 30, 2026 02:52
@chrisb13
Copy link
Copy Markdown
Collaborator Author

@atteggiani any idea why the RTD preview isn't showing up here? (Or am I just missing the button somehow?!)

It's over here for example.

@dougiesquire did you want to review this or would you rather it just gets merged? (I'm easy either way.)

@atteggiani
Copy link
Copy Markdown
Collaborator

@atteggiani any idea why the RTD preview isn't showing up here? (Or am I just missing the button somehow?!)

Not sure what you mean.
I can see the RTD preview by clicking on the RTD build check:

Screenshot 2026-04-30 at 1 40 14 pm

Comment thread documentation/docs/pages/season1.md Outdated
Comment thread documentation/docs/pages/season1.md Outdated
Comment thread documentation/docs/pages/season1.md Outdated
Comment thread documentation/docs/pages/season1.md Outdated
Comment thread documentation/docs/pages/season1.md Outdated
Comment thread documentation/docs/pages/season1.md Outdated
applying @dougiesquire's review suggestions

Co-authored-by: Dougie Squire <dougie.squire@anu.edu.au>
@chrisb13
Copy link
Copy Markdown
Collaborator Author

chrisb13 commented Apr 30, 2026

Not sure what you mean.
I can see the RTD preview by clicking on the RTD build check:

Oh so odd, it's working fine for me now too. Sorry for the false alarm.

@dougiesquire thanks for the review, and correcting my many typos. I'll merge now but do let me know if anything is amiss.

@chrisb13 chrisb13 merged commit edcc30a into main Apr 30, 2026
3 of 4 checks passed
@chrisb13 chrisb13 deleted the Part2-code-back2MOM6 branch April 30, 2026 04:15
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.

3 participants