~edwardloveall/scribe

Re: [PATCH] Add Dockerfile

Details
Message ID
<D09E14F2-A138-4F02-8911-CEC4F78487F0@edwardloveall.com>
DKIM signature
pass
Download raw message
Hello! Thanks for your patch and interest in Scribe 😃 Your are 100% 
correct about there being no patch email. I have just fixed that and 
added a mailing list. If I'm understanding it correctly, this might be 
the first email on the list!

You mentioned that this Docker configuration isn't fully tested yet. 
Also, based on the patch, it seems like this is intended for use in 
development. Is that correct? My personal experience with Docker is 
mixed at best. I think it can work well for deployment (although it's 
not what I use) or if you don't need to interact with what's inside the 
Docker container very often. An example might be a server processing 
background jobs or sending emails that runs alongside the app you're 
developing.

I haven't had success when using it for development. It has a big memory 
footprint and all commands I want to run are now one step removed. I'm 
concerned that if I don't use Docker it may break and I won't know, nor 
will I be motivated to fix it. I'm sure someone could come along and fix 
it eventually so I'm less concerned about this, but it's still not an 
ideal experience. 😕

To be clear, I think it's fine if people want to use Docker during 
development, even on this project. To each their own! I'd even be happy 
to link to Docker configs in the README. But I'm concerned about adding 
and supporting it officially.

Can you tell me more about the problems you're experiencing that 
motivated this patch?

Edward

P.S. I have no idea who you are, kind person that submitted the patch. 
There doesn't seem to be any name or handle attached to this change! 
Maybe that's on purpose but if not I thought you'd like to know. 🙂

On 14 Oct 2021, at 5:42, acc-3npsourcehut21@remerge.net wrote:

> Hello,
>
> Thanks for a neat project!
>
> I haven't fully tested this yet, but it's based on the official 
> example
> from the lucky project.
>
> BTW, would be helpful if the intended recipient for patch e-mail was
> listed in the repo (right now it's just refering your contact details 
> to
> your website).
Reply to thread Export thread (mbox)