Fix docs site build #4
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
mhausenblas/mkdocs-deploy-gh-pages needs a requirements file, else it tries to install the repo-level requirements, which typically are not needed for the docs site, see: mhausenblas/mkdocs-deploy-gh-pages#53 introduce requirements file specific to the docs site, fixing an issue with trying to install the dev-requirements.txt the underlying issue was with trying to install black into the Docker image used by this GitHub Action (squidfunk/mkdocs-material:7.1.8), see: https://github.com/mhausenblas/mkdocs-deploy-gh-pages/blob/master/Dockerfile recent versions of black depend on regex, which has a history of trouble with Linux wheel availability and support * psf/black#1207 * https://bitbucket.org/mrabarnett/mrab-regex/issues/343/wheel-for-linux * https://bitbucket.org/mrabarnett/mrab-regex/issues/349/no-module-named-regex_regex-regex-is-not-a the Docker image does not have gcc available; rather than installing a build chain to build regex to enable the install of black, which isn't even used by the docs site, the simpler fix was to create a docs/requirements.txt and point the GitHub Action at it instead similar report on the Action repo: mhausenblas/mkdocs-deploy-gh-pages#105
|
Man - why does black need a separate regexer - ug. |
|
An interesting anecdote: dials/dials#1585 (comment) |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
mhausenblas/mkdocs-deploy-gh-pagesneeds a requirements file, else it tries to install the repo-level requirements, which typically are not needed for the docs site, see: mhausenblas/mkdocs-deploy-gh-pages#53This PR introduces a requirements file specific to the docs site, fixing an issue with trying to install the dev-requirements.txt.
The underlying issue was with trying to install black into the Docker image used by this GitHub Action (squidfunk/mkdocs-material:7.1.8), see: https://github.com/mhausenblas/mkdocs-deploy-gh-pages/blob/master/Dockerfile
Recent versions of black depend on regex, which has a history of trouble with Linux wheel availability and support
The Docker image does not have gcc available; rather than installing a build chain to build regex to enable the install of black, which isn't even used by the docs site, the simpler fix was to create a docs/requirements.txt and point the GitHub Action at it instead.
Similar report on the Action repo: mhausenblas/mkdocs-deploy-gh-pages#105