Skip to content

Try parallel Sphinx builds and hide progress #13241

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Feb 28, 2020
Merged

Conversation

wouterj
Copy link
Member

@wouterj wouterj commented Feb 24, 2020

Let's see if we can make Sphinx faster this way :)

@smoench
Copy link
Contributor

smoench commented Feb 25, 2020

I also tried this before and got the same result. But I wasn't able to fix https://github.com/fabpot/sphinx-php

@wouterj wouterj force-pushed the wouterj-sphinx-github-action branch from 0918f6d to 502e443 Compare February 25, 2020 11:52
@wouterj
Copy link
Member Author

wouterj commented Feb 25, 2020

@smoench you're right, I've submitted a "fix" to the sphinx extensions: fabpot/sphinx-php#43

@OskarStark
Copy link
Contributor

I restarted the build

@smoench
Copy link
Contributor

smoench commented Feb 27, 2020

The commit hash for sphinx-php in _build/.requirements.txt needs to be updated :-)

@wouterj wouterj force-pushed the wouterj-sphinx-github-action branch from 502e443 to 997d0e3 Compare February 27, 2020 20:36
@wouterj wouterj force-pushed the wouterj-sphinx-github-action branch from 997d0e3 to b80765a Compare February 28, 2020 12:18
@wouterj
Copy link
Member Author

wouterj commented Feb 28, 2020

Sadly, seems like Sphinx v1.4.9 has a bug with parallel runs (the feature was also marked as "experimental in 1.4"). So back to non-parallel runs, but now with a less verbose output in the GitHub Action.

@wouterj wouterj merged commit 63c92a1 into 3.4 Feb 28, 2020
@wouterj wouterj deleted the wouterj-sphinx-github-action branch February 28, 2020 12:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants