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

Update README.md, remove README_TODO.txt #425

Merged
merged 8 commits into from
Jan 7, 2022

Conversation

mkavulich
Copy link
Collaborator

Update README.md, remove README_TODO.txt

Updating README.md to remove badly outdated information and add links to existing documentation elsewhere.
Also, remove README_TODO.txt which is obsolete.

User interface changes?: No

Fixes: Partially addresses Github issue #400, we will need to decide whether capgen-specific documentation should go in the README or elsewhere before I'd consider that issue fully resolved.

Testing:
None, edits to README files only.

Copy link
Collaborator

@gold2718 gold2718 left a comment

Choose a reason for hiding this comment

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

Thank you!!

Copy link
Collaborator

@climbfuji climbfuji left a comment

Choose a reason for hiding this comment

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

Yes, thank you very much indeed!

@grantfirl we can either merge this right away and use one of the next CCPP PRs to advance the submodule pointer in fv3atm, or we can wait and follow the usual procedure. But this is completely safe to merge now.

@grantfirl
Copy link
Collaborator

Yes, thank you very much indeed!

@grantfirl we can either merge this right away and use one of the next CCPP PRs to advance the submodule pointer in fv3atm, or we can wait and follow the usual procedure. But this is completely safe to merge now.

Sure, let's merge now and I'll update the submodule pointer as part of the latest Thompson tuning ccpp-physics PR that will go in next week. I'm going to be doing the upstream PRs on that, so we won't need others' cooperation.

@climbfuji climbfuji merged commit e7ecb34 into NCAR:main Jan 7, 2022
@grantfirl
Copy link
Collaborator

FYI, updated submodule pointer is here: NOAA-EMC/fv3atm#463

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

Successfully merging this pull request may close these issues.

4 participants