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

feat : custom repositories #2248

Open
wants to merge 91 commits into
base: main
Choose a base branch
from
Open

feat : custom repositories #2248

wants to merge 91 commits into from

Conversation

jokestax
Copy link
Contributor

we can now deploy multiple repos on same organistation using below flags :

"--gitopsRepoName" metaphorRepoName "adminTeamName" "developerTeamName"

but we have to use kubefirst-api repo branch "feat-custom-repo" and kubefirst-gitops-template repo branch "feat-custom-repo" until its merged into main

@mrsimonemms
Copy link
Contributor

Change looks good, but a couple of nits:

  • can you run go fmt ./... as the formatting is all out on the changes (this may change additional files which shouldn't be updated here - just the lines you've changed)
  • there's a merge conflict which needs resolving

@fharper fharper changed the title feat : custom repo for civo feat : custom repositories Jul 23, 2024
@fharper
Copy link
Contributor

fharper commented Jul 23, 2024

@jokestax can you fix the conflict before we review this please?

@jokestax
Copy link
Contributor Author

jokestax commented Jul 23, 2024

I fixed it in the recent push

@fharper
Copy link
Contributor

fharper commented Jul 23, 2024

We won't be able to merge this PR, you need to fix the conflict at the Git level.

CleanShot 2024-07-23 at 11 32 51@2x

@jokestax jokestax marked this pull request as draft July 23, 2024 16:46
@jokestax
Copy link
Contributor Author

jokestax commented Jul 23, 2024

@fharper, done (k3d isnt supported yet)

@jokestax jokestax self-assigned this Jul 23, 2024
@fharper
Copy link
Contributor

fharper commented Jul 24, 2024

@jokestax you wrote done, but it's a draft, just want to be sure it's ready for review? :)

@fharper
Copy link
Contributor

fharper commented Jul 24, 2024

oh, should I wait for k3d support?

@jokestax
Copy link
Contributor Author

jokestax commented Jul 24, 2024 via email

@fharper
Copy link
Contributor

fharper commented Jul 24, 2024

Nice, undraft this PR when you are ready for us to review it :)

@jokestax jokestax marked this pull request as ready for review July 24, 2024 20:02
@jokestax
Copy link
Contributor Author

its ready 👍

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.

7 participants