common
common copied to clipboard
readthedocs
Shared bits around multiple repositories
acc68e51 added the ability to customize RTD_PRODUCTION_DOMAIN using the --http-domain argument to docker.up The Docker Compose settings also allows RTD_PUBLIC_DOMAIN to be configured using an env var but this appears...
This is similar to what we are doing in PostgreSQL to avoid running long queries and degrading the service. Related https://github.com/readthedocs/addons/issues/84
Those files are copied from other sources or edited by other tools and IMHO not subject for pre-commit handling. Refs: - https://github.com/readthedocs/readthedocs.org/issues/11200 - https://github.com/readthedocs/readthedocs.org/pull/11199
For - https://github.com/readthedocs/readthedocs.org/pull/11195 custom exclude needed due to - https://github.com/codespell-project/codespell/issues/3196
Remove a bunch of these following lines when modifying a file. ``` proxito_1 | [2024-03-06 15:32:58 +0000] [22] [ERROR] Worker (pid:23) was sent SIGTERM! proxito_1 | [2024-03-06 15:32:58 +0000] [22]...
I found there is an option to remove the initial banner in Celery, but I'm not sure how to pass that argument. See https://github.com/celery/celery/issues/3496 ``` build_1 | Please specify a...
pyinvoke 1.7.1 fails with Python 3.11 due to changes to inspect.getargspec in the standard library. This has been fixed in 2.2.0 A smoke test of `inv docker.up` and `inv docker.down`...
I'd like to propose a couple of small tweaks to `inv docker.up` RTD_DJANGO_DEBUG has a default value of True, so ensure that an empty variable is passed to docker compose...
The idea would be to call `inv docker.up --addons-local-dev` or similar and make the CF worker to inject `http://localhost:8000/readthedocs-addons.js` instead of the `/_/static/` URL. See https://github.com/readthedocs/readthedocs.org/pull/10965#discussion_r1479728044 for more context.
I'm assuming we shouldn't have checks that conflict with Black, so removing this.
