-
Notifications
You must be signed in to change notification settings - Fork 42
Open
Labels
ImprovementMinor improvement to codeMinor improvement to code
Description
Currently our pre-commit settings for prospector doesn't really work well.
Can't be re-used in .com, we had to disable that check https://github.com/readthedocs/readthedocs-corporate/pull/1698. The reason it fails is because of two things:
- The Python path: when running on CI, we clone the readthedocs.org repo in the .tox/py310/readthedocs.org directory, but in dev, we have this repo in the upper directory (../readthedocs.org).
- Requirements: prospector creates a separate environment to run in, due to this, we need to install the requirements inside that environment, and since we are sharing this file for both of our repos, we ended up installing the requirements from a URL, this is a problem if a new dep is added in a PR, or if .com needs an additional dep that isn't on .org.
Line 101 in e598102
- https://raw.githubusercontent.com/readthedocs/readthedocs.org/main/requirements/testing.txt
Metadata
Metadata
Assignees
Labels
ImprovementMinor improvement to codeMinor improvement to code