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

[WIP] Major cleanup of zm_conv_intr.F90 #6850

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

whannah1
Copy link
Contributor

This is the inaugural PR for the ZM overhaul and refactoring, which is limited to cosmetic cleanup of the outer ZM interface (zm_conv_intr.F90).

[BFB]

@whannah1 whannah1 added Atmosphere BFB PR leaves answers BFB labels Dec 13, 2024
@whannah1
Copy link
Contributor Author

For reviewers - looking at the diffs is going to make your head explode 🤯 , so I recommend just looking at the final modified file instead. I touched pretty much every line and the vast majority of changes are white space adjustments.

Copy link

PR Preview Action v1.4.8
🚀 Deployed preview to https://E3SM-Project.github.io/E3SM/pr-preview/pr-6850/
on branch gh-pages at 2024-12-13 21:36 UTC

@whannah1
Copy link
Contributor Author

e3sm_atm_integration passes on chrysalis

@whannah1 whannah1 added the ZM label Dec 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Atmosphere BFB PR leaves answers BFB ZM
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants