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

RFC: core-lib-repo #4

Merged
merged 2 commits into from
May 19, 2021
Merged

RFC: core-lib-repo #4

merged 2 commits into from
May 19, 2021

Conversation

applejag
Copy link
Contributor

@applejag applejag commented May 12, 2021

Proposed location to place utility code, such as loading config values or
serving version endpoints, so that our other repositories can take use of it.

Rendered: https://github.com/iver-wharf/rfcs/blob/90fdcad/_published/0004-core-lib-repo.md

@applejag applejag added the rfc This issue or pull requests contains a new Request For Comments label May 12, 2021
@applejag applejag self-assigned this May 12, 2021
@applejag applejag added this to In progress in Backlog via automation May 12, 2021
@applejag applejag force-pushed the rfc/core-lib-repo branch 2 times, most recently from 47bc489 to d016b1f Compare May 12, 2021 12:54
@fredx30
Copy link
Contributor

fredx30 commented May 14, 2021

My gut feeling when downloading the repos was that this could use a master repo that defines the others as subrepos. This could potentially allow for a recursive clone of the master repo as well as providing an outer path where shared resources could be stored. Another such resource that comes to mind is the docker-compose repo/file.
Edit: See ref here https://git-scm.com/book/it/v2/Git-Tools-Submodules

Copy link
Contributor

@Pikabanga Pikabanga left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 for creating a wharf-core repo

_published/0004-core-lib-repo.md Show resolved Hide resolved
_published/0004-core-lib-repo.md Outdated Show resolved Hide resolved
_published/0004-core-lib-repo.md Show resolved Hide resolved
Backlog automation moved this from In progress to Reviewer approved May 18, 2021
@applejag
Copy link
Contributor Author

@fredx30
My gut feeling when downloading the repos was that this could use a master repo that defines the others as subrepos. This could potentially allow for a recursive clone of the master repo as well as providing an outer path where shared resources could be stored. Another such resource that comes to mind is the docker-compose repo/file.
Edit: See ref here https://git-scm.com/book/it/v2/Git-Tools-Submodules

Submodules is a fine thought. What you're proposing is a switch in how we work with the repos locally, where each local repo is a submodule of the "main" repo, if I understand you correctly.

Pros:

  • Only 1 repo to clone, instead of 6 distinct. Simpler onboarding procedure, although that is just a one-time thing each developer has to do and we do have it documented by now.
  • 1 fewer repos, given that we put the core lib in this "main" repo
  • No need to symlink the docker-compose.yml file to outside of the repo it's living in

Cons:

  • We have to make sure the main repo is up to date all of the time with its submodule references
  • Prone to confusion as we're versioning all the repos separately while the main repo that includes all the other repos has its own version of only the utility code inside itself.

Given these pros and cons, I personally would rather see the wharf-docker-compose repo have submodules and have some automatic script/GitHub action that updates its submodules whenever there's an update, instead of mixing this with the core/utility lib repo.

@applejag
Copy link
Contributor Author

I'm finalizing the name as wharf-core

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rfc This issue or pull requests contains a new Request For Comments
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

4 participants