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

Extract mksi into new repo #157

Merged
merged 3 commits into from
Sep 24, 2024
Merged

Extract mksi into new repo #157

merged 3 commits into from
Sep 24, 2024

Conversation

mathomp4
Copy link
Member

@mathomp4 mathomp4 commented Feb 9, 2024

This PR along with GEOS-ESM/GEOSadas#267 is an attempt to extract the mksi folder in GEOSana_GridComp into its own repo. To do so, the RepoExtractor tool was run as:

./extract_repo.bash -r "GEOSana_GridComp" -d "mksi" --newrepo "GEOS_mksi" --develop --create-repo --push

and a new repo GEOS_mksi was created:

https://github.com/GEOS-ESM/GEOS_mksi

Then GEOS_mksi was added as a new component and placed as @mksi/ under GSI_GridComp/

Tests show it builds which should be proof it is correct, but probably needs testing to be sure.

And of course, before anything is done, tags should be made in GEOS_mksi and GEOSana_GridComp.

Closes GEOS-ESM/swell#289

@mathomp4 mathomp4 added the 0 diff structural Structural changes to repository that are zero-diff label Feb 9, 2024
@mathomp4 mathomp4 self-assigned this Feb 9, 2024
rtodling
rtodling previously approved these changes Sep 24, 2024
@rtodling rtodling marked this pull request as ready for review September 24, 2024 12:09
@rtodling rtodling requested review from a team as code owners September 24, 2024 12:09
@rtodling
Copy link
Contributor

@mathomp4 Have Matt, can we just get this in?

@mathomp4
Copy link
Member Author

@mathomp4 Have Matt, can we just get this in?

Oh sure. Let me fix the conflicts and then you can merge.

# Conflicts:
#	GEOSaana_GridComp/GSI_GridComp/mksi/sidb/active_channels.tbl
#	GEOSaana_GridComp/GSI_GridComp/mksi/sidb/available_channels.tbl
#	GEOSaana_GridComp/GSI_GridComp/mksi/sidb/iuseClass-4_channels.tbl
@mathomp4 mathomp4 changed the title WIP: Extract mksi into new repo Extract mksi into new repo Sep 24, 2024
@mathomp4
Copy link
Member Author

@rtodling I fixed up the conflicts and it should be good to go. You can re-approve and merge as you wish.

Copy link
Contributor

@rtodling rtodling 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.

@rtodling rtodling merged commit b3b697d into develop Sep 24, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0 diff structural Structural changes to repository that are zero-diff
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Split GSI's mksi into its own repo
3 participants