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

MNT: Convert cxi to submodule #2

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

Conversation

ZryletTC
Copy link
Contributor

@ZryletTC ZryletTC commented Aug 29, 2022

Change of approach from #1
First step of splitting this repo into multiple smaller repos. Replacing each subdirectory with a git submodule linking to the new repositories. This is done following these instructions: https://coderwall.com/p/a3a5xg/splitting-a-project-sub-directory-to-a-new-git-repo

The new cxi repo is here: https://github.com/pcdshub/cxi-config
When this PR is merged, I'll pull it into /cds/group/pcds/config and init the submodule there.

If all goes well, this process will be repeated for all other hutches as well (and probably the non-hutches too?)

Currently testing...
I was concerned this might disrupt iocmanager's commits but it looks like it doesn't do any git pushes so it shouldn't care.

@klauer
Copy link
Contributor

klauer commented Aug 29, 2022

(More context in #1)
I think that before we go any further with this, we should decide if cxi-config and even epics-config are the appropriate names here.

Ideally, these would be specific enough to know what's in the configuration repository and disambiguate them the hutch-python-specific config which is a bare "cxi" https://github.com/pcdshub/cxi/

It may not seem a big deal now, but this is likely to cause confusion down the line once we have 10 hutch and 10 hutch-config repos...

@ZryletTC
Copy link
Contributor Author

I agree, not sure on good names though. Something like apps-config-cxi?

@mcb64
Copy link
Contributor

mcb64 commented Aug 30, 2022

I like the sound of apps-config-HUTCH.

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

Successfully merging this pull request may close these issues.

3 participants