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

Make the model configuration sections more consistent #449

Merged
merged 8 commits into from
Jul 19, 2023

Conversation

MartinDix
Copy link
Contributor

Make the AM, ESM, CM and OM model configuration pages more consistent.

Will close #400, #401, #402 and #447.

@github-actions
Copy link

github-actions bot commented Jul 19, 2023

PR Preview Action v2.1.1
🛬 Preview removed because the pull request was closed.
2023-07-19 06:57 UTC

@MartinDix MartinDix requested a review from heidinett July 19, 2023 00:59
Copy link
Contributor

@heidinett heidinett left a comment

Choose a reason for hiding this comment

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

Looks good Martin. I directly edited minor text edits and typos in the respective files and committed them to your respective branch.
The following link in ACCESS-AM is broken: CSIRO ACCESS-CM2 configuration. Either fix it or remove the link.

Copy link
Contributor

@heidinett heidinett left a comment

Choose a reason for hiding this comment

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

Thanks Martin!

@heidinett heidinett merged commit dbd817a into development Jul 19, 2023
2 checks passed
@MartinDix
Copy link
Contributor Author

Too soon unfortunately, the link is still broken. I'll fix and make a new PR.

@heidinett
Copy link
Contributor

Looks good Martin - link works!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants