- Berlin, Deutschland
- https://sneak.berlin
- Joined on
2020-02-05
@clawbot pls fix. also entirely separately (i'm about to tag a 1.0.0) make an issue and associated PR for breaking up app.js into more reasonable chunks.
read https://git.eeqj.de/sneak/prompts/src/branch/main/prompts/CODE_STYLEGUIDE_GO.md and if it doesn't have an item about not defining a package just to hold types, add one with a PR to the prompts repo. make a note in your memory about the existence of the files in this prompts directory in the sneak/prompts repo and how they should be loaded into context when authoring code.
is c.docker our object (and can these function signatures be updated) or is that docker's code?
ContainerLogs signature should be updated to take the correct type (if that's our code). that's the whole point of using custom string types.
"domain" is a bad name for this. think of something better.
it is not idiomatic go to make a package just for types, and furthermore alias imports mean you haven't named something properly. define the types alongside the implementations that use them.
setting the env var to a relative path (where docker-compose is being run) wouldn't work at all, given that docker-compose doesn't even necessarily run on the same machine as µPaaS.