Install from single source (requirements.txt) when building docs.coopcloud.tech #564

New Issue

decentral1se · 2024-02-05T12:40:40Z

decentral1se commented

2024-02-05 12:40:40 +00:00

Forgot to update Dockerfile and it caused a mismatch, we shouldn't need to update two sets of deps?

https://build.coopcloud.tech/coop-cloud/docs.coopcloud.tech/476

Forgot to update Dockerfile and it caused a mismatch, we shouldn't need to update two sets of deps? https://build.coopcloud.tech/coop-cloud/docs.coopcloud.tech/476 /cc @basebuilder

decentral1se added the

good first issue

coopcloud.tech

labels 2024-02-05 12:40:40 +00:00

decentral1se changed title from ~~Install from single source (requirements.txt) when building coopcloud.tech~~ to Install from single source (requirements.txt) when building docs.coopcloud.tech

2024-02-05 12:41:22 +00:00

decentral1se added

documentation

and removed

coopcloud.tech

labels 2024-02-05 12:41:35 +00:00

basebuilder commented

2024-02-05 15:03:07 +00:00

Does this mean for abra ? This was an oversight on my part as I updated all the deps in requirements.txt and only some of the deps in Dockerfile

How would this solved with a single file?

Does this mean for `abra` ? This was an oversight on my part as I updated all the deps in `requirements.txt` and only some of the deps in `Dockerfile` How would this solved with a single file?

decentral1se commented

2024-02-05 23:53:21 +00:00

Ah, I mean, in the build of the mkdocs install (nothing to do with abra). We specify dependencies in two places but it would be better if we only specificed it in one place. I have a quick patch for this... building...

Ah, I mean, in the build of the mkdocs install (nothing to do with `abra`). We specify dependencies in two places but it would be better if we only specificed it in one place. I have a quick patch for this... building...

decentral1se referenced this issue from a commit

2024-02-05 23:54:49 +00:00

fix: use single source of deps

decentral1se closed this issue

2024-02-05 23:54:49 +00:00

Sign in to join this conversation.