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

fix documentation #206

Merged
merged 1 commit into from
Dec 29, 2022
Merged

fix documentation #206

merged 1 commit into from
Dec 29, 2022

Conversation

uturuncoglu
Copy link
Collaborator

Description of changes

This PR aims to fix CDEPS documentation source

Specific notes

Contributors other than yourself, if any: @jedwards4b

CDEPS Issues Fixed (include github issue #): None

Are there dependencies on other component PRs (if so list): No

Are changes expected to change answers (bfb, different to roundoff, more substantial):
No. It changes only the documentation.

Any User Interface Changes (namelist or namelist defaults changes):
No

Testing performed (e.g. aux_cdeps, CESM prealpha, etc):
Not required

Hashes used for testing:
N/A

@uturuncoglu uturuncoglu mentioned this pull request Dec 29, 2022
@jedwards4b
Copy link
Contributor

I can create the cdeps1.0.0 tag when these are merged.

@uturuncoglu
Copy link
Collaborator Author

@jedwards4b Okay. Then, I could update the compiled source PR to have cdeps1.0.0 tag and we could mer that one too. Sorry for lots of work. Updating documentation seems very time consuming. Maybe we could create an GH action for it in the future.

@uturuncoglu
Copy link
Collaborator Author

If it is okay for you, we could merge it now.

@jedwards4b
Copy link
Contributor

Go ahead

@uturuncoglu uturuncoglu merged commit eb287f7 into ESCOMP:master Dec 29, 2022
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