-
Notifications
You must be signed in to change notification settings - Fork 31
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
docky init simplification #155
Comments
Hello I would also recommend looking at how does Doodba bootstrap new projects as well because they do lot's of things pretty well too, I'll try to look today... |
so Doodba does it all with copier https://github.com/Tecnativa/doodba-copier-template |
If you want simplify, please remove secrets.docker-compose.yml |
I agree, this would simplify a lot of things just for a copier command. We could name our fork of copier |
or copipy ? |
So let's define the objectives of the project. Who is it for? What it should do ? What techno it should support (podman, docker compose v2) ? etc. Then we will know what to put in the scope. |
For me The objectif of docky is to have a make simplier the use of docker-compose
It can be use for Odoo of whatever project, but project that respect the docky logic: We can replace docky by a big list of alias on top of docker-compose but it will be complex bash code. For podman as there is not "docker-compose" like project I prefer to continue with docker- compose and so we should move to docker-compose 2. (We can add it in the roadmap). But indeed we need to take time to define and explain in the readme the aim and roadmap. |
Note: I am not close-mind In any case, if we want to conserve the templating stuff, I think we should improve the command and having something like docky project init => create .env file (after a git clone of an existing project) So I propose |
change have been done here : #154 |
just to make it clear: to test it:
(yes I needed to set |
IMO: Who is it for : What is it used for:
And less often, but because is rarely done, having a simple way of doing multiple steps is good:
Docky is not mandatory; it's just a wrapper around some other tools. More details about commands: Easy, no changes More nuanced:
I'm not sure our approach is good here. We use this interactive work inside with starting with bash because our images are too long to start.
About rare tasks:
Benefits:
Benefits:
Benefits:
Benefits:
|
Thanks @hparfr for this full time consuming analyze. |
Just a remark: I think an important use case is also to give our IT literate customers an easy way to get a dev replica of their project up and running. In many cases it's also a condition for them to trust us and our hosting. In this case they won't need the init command. But having docky easy to install and look standard may help. |
I agree with @bealdav Maybe the easiest path is to simplify docky enough to be able to replace it with a properly documented bash file with only aliases defined inside. That way you can either install the file directly in your bash_aliases or you can read the online version and paste the command you need. The we can restrict the docky python code to more complex tasks like docky init. |
Not than abbreviation in fish shell are really more powerful than alias in bash because it replace shortcut by real cmd including resolving var eval. |
Update to my previous message:
Still true.
Docky in prod is not an hard requirement, the command used in prod are:
It think now, we should put these two features (scaffolding & initialization) in another tool.
Still true
Still true
For the moment, we don't use it on CI.
|
Hi
I would like to simplify docky init.
It should only generate the ".env" file and nothing more.
As docky can be use by whatever project (I use for node and ruby). I prefers to keep it simple as possible.
If we want to generate a new project I prefer to use directly copier (we do not create a lot of projet).
And copier is not something hard to use and it also good to know how to use it, as sometime you need to launch a copy update on existing OCA Repo
@bealdav @PierrickBrun @hparfr
This will solve the publication of docky on pipy and also unblock the docky-template
The text was updated successfully, but these errors were encountered: