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

merge BRIDGE into main in preparation for release v1.0.2 #33

Closed
wants to merge 28 commits into from

Conversation

gmao-rreichle
Copy link
Collaborator

No description provided.

weiyuan-jiang and others added 22 commits March 29, 2024 17:10
Currently, GEOSgcm_GridComp is cloned via mepo under GEOSldas_GridComp (which echoes how it
was in in GEOSldas v17 and older). This PR moves GEOSgcm_GridComp under Components in GEOSldas. This
makes its location consistent with GEOSgcm and GEOSldas.

Note that this PR requires a corresponding change in the XXX repo:
Remove `GEOSgcm_GridComp` references in `.gitignore`
@biljanaorescanin
Copy link
Collaborator

@mathomp4 For some reason we have errors in build here

@mathomp4
Copy link
Member

@mathomp4 For some reason we have errors in build here

Yeah. I need to fix/change some stuff in the LDAS CI. You guys do things in a way that can break my CI. Give me a bit to puzzle over this.

weiyuan-jiang
weiyuan-jiang previously approved these changes Apr 12, 2024
@weiyuan-jiang weiyuan-jiang self-requested a review April 12, 2024 13:33
@gmao-rreichle
Copy link
Collaborator Author

Closing this PR because we should be able to go directly from develop to main on GEOSldas_GridComp. There is no need to edit a components.yaml file, as we need to do on the GEOSldas fixture.

@gmao-rreichle gmao-rreichle deleted the BRIDGE branch April 12, 2024 14:00
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.

5 participants