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.

Sign up for GitHub

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

Jump to bottom

Move MAM, MATRIX, DNA, CARMA, GAAS, and GEOSachem to separate repos #260

Closed
wants to merge 10 commits into from
Closed

Move MAM, MATRIX, DNA, CARMA, GAAS, and GEOSachem to separate repos #260

wants to merge 10 commits into from

Conversation

mathomp4
Copy link
Member

@mathomp4 mathomp4 commented May 2, 2023 •
edited
Loading

This PR is needed for GEOS-ESM/GEOSgcm#571 for the effort to remove MAM, MATRIX, DNA, CARMA, GAAS, and GEOSachem to separate repos.

Of course, this can only be taken in concert with changes to GEOSgcm to clone the new repos.

NOTE: This will fail CI due to the GEOSgcm it clones not knowing about the new repos.

@mathomp4 mathomp4 added Contingent - DNA Do Not Approve (DNA). These changes are contingent on other PRs 0 diff The changes in this pull request have verified to be zero-diff with the target branch. labels May 2, 2023
@mathomp4 mathomp4 self-assigned this May 2, 2023
@github-actions
Copy link

github-actions bot commented May 2, 2023

Label error. Requires exactly 0 of: Contingent - DNA, Needs Lead Approval, Contingent -- Do Not Approve. Found: Contingent - DNA, 0 diff

@mathomp4 mathomp4 mentioned this pull request May 2, 2023
Closed
16 tasks
@github-actions
Copy link

github-actions bot commented May 2, 2023

Label error. Requires exactly 0 of: Contingent - DNA, Needs Lead Approval, Contingent -- Do Not Approve. Found: Contingent - DNA, 0 diff

1 similar comment
@github-actions
Copy link

github-actions bot commented May 2, 2023

Label error. Requires exactly 0 of: Contingent - DNA, Needs Lead Approval, Contingent -- Do Not Approve. Found: Contingent - DNA, 0 diff

@github-actions
Copy link

Label error. Requires exactly 0 of: Contingent - DNA, Needs Lead Approval, Contingent -- Do Not Approve. Found: Contingent - DNA, 0 diff

@mathomp4 mathomp4 changed the title Move MAM, MATRIX, and DNA to separate repos Move MAM, MATRIX, DNA, CARMA, GAAS, and GEOSachem to separate repos May 25, 2023
@github-actions
Copy link

Label error. Requires exactly 0 of: Contingent - DNA, Needs Lead Approval, Contingent -- Do Not Approve. Found: Contingent - DNA, 0 diff

1 similar comment
@github-actions
Copy link

Label error. Requires exactly 0 of: Contingent - DNA, Needs Lead Approval, Contingent -- Do Not Approve. Found: Contingent - DNA, 0 diff

@mathomp4
Merge branch 'develop' into feature/mathomp4/separate-dna-mam-matrix
bf7aeaf 
# Conflicts:
#	CHANGELOG.md
#	GAAS_GridComp/LDE_Mod.F90
#	GEOS_ChemGridComp.F90
@github-actions
Copy link

github-actions bot commented Aug 8, 2023

Label error. Requires exactly 0 of: Contingent - DNA, Needs Lead Approval, Contingent -- Do Not Approve. Found: Contingent - DNA, 0 diff

@mathomp4
Merge branch 'develop' into feature/mathomp4/separate-dna-mam-matrix
92fdb94 
# Conflicts:
#	CARMAchem_GridComp/CARMA_GridComp.F90
#	CARMAchem_GridComp/CARMA_UtilMod.F90
#	CARMAchem_GridComp/CARMAchem_GridCompMod.F90
#	GAAS_GridComp/GAAS_Registry.rc
#	GEOS_ChemGridComp.F90
@github-actions GitHub Actions
Copy link

Label error. Requires exactly 0 of: Contingent - DNA, Needs Lead Approval, Contingent -- Do Not Approve. Found: Contingent - DNA, 0 diff

@mathomp4 mathomp4 mentioned this pull request Jan 23, 2025
Merged
@mathomp4
Copy link
Member Author

Close in favor of #291

@mathomp4 mathomp4 closed this Jan 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@mathomp4 mathomp4

Labels
0 diff The changes in this pull request have verified to be zero-diff with the target branch. Contingent - DNA Do Not Approve (DNA). These changes are contingent on other PRs
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@mathomp4