Open pmorie opened 6 years ago
Previously I had copied the files from this repo into my existing one. Now I am starting over, with the idea that I will rebase to keep in sync. Quickly realized that rebase isn't going to work with my own project goals because I wanted to:
I think with the amount of files that I'd be changing/renaming, it would be easier to consume upstream changes with merges instead. My gut says that will be less work for me to manage the inevitable conflicts that arise, better tracking of renames, etc. I'll update this comment with how that goes and share what workflow ends up working for me. 😀
Alternatively, have you considered making this a library + a starter repo? That way you could make all sorts of changes to the library without worrying about merge conflicts, and people could continue to use the starter repo as a jumping off point, but there would be no expectation that they would avoid modifying any of the files in that starter repo.
Alternatively, have you considered making this a library + a starter repo?
@carolynvs Yes, it was something I was thinking about as well. We could, if possible have the rest
and broker
pkgs in a separate repo, and leave all the other things in this repo as an example in a similar way as they do in the k8s sample-controller.
There needs to be one small change to separate the user dependancy in the rest
pkg, but that is easy enough, see here -> https://github.com/pmorie/osb-starter-pack/commit/e909008840314d88fab12448b9cd3b3f99c578ee.
@pmorie If you agree with the above, could those pkgs be something that could be incorporated into the go-open-service-broker-client?
I would rather not put tools for building a broker directly into the client, how about layering into a broker library and then consuming that here? Is that getting too complex? Maybe I should think about this a little further.
how about layering into a broker library and then consuming that here?
That sgtm as well. Think having a broker lib will make it easier to change things, as @carolynvs mentioned above.
We should have: