Add docker and don't push build#27
Add docker and don't push build#27ParkerMc wants to merge 6 commits intoOpenMaxIO:openMaxIO-mainfrom
Conversation
Update README.md
|
while looking strange at first by having the build folder in the repo , it has the purpose so can use go build without having to build the frontend first, like this go install github.com/OpenMaxIO/openmaxio-object-browser/cmd/console@lateststill you would have to have an eye on not committing changes on the build folder, except its an release commit for the container look at my comment at #26 (comment) |
|
Yeah, this is the common problem when using multiple different tool chains in the same project. I lean much more towards keeping those out of git because the repository is already pretty large when you clone it as the whole build folder is basically replaced every release. I can also see plenty of people(including me) accidentally committing the build folder. Of course this is a cost benefit trade off, I don't see too much of a benefit to allow using go install, but you may see something I'm missing. I think the comment you have at #26 (comment) is great for docker only release, but due to the number release types(deb, binary, docker) it would probably be better to have the same binary across the board, this also prevents a go build change from needing to be changed in many places. |
|
It's also a Go library, in case you want to embed console in the minio binary. While I may agree about size, it would be interesting how large the history for only the build folder alone is, 700MB for the entire git repo didn’t look that large to me, but code size only is rather small. The dockerfile is not for releases, as for that a different one is used, that also just copies the binary and goreleaser takes care of that. |
Addresses: #4, #5, #8, #13
In order to merge this first the main branch needs to be change to not have any changes past when they removed all the functionality. I'll make a follow up PR shortly to add the updates they applied.
Here's what I did to make that branch here https://github.com/ParkerMc/openmaxio-object-browser/tree/main
I recommend doing the same and then changing the default to this new main branch.