Skip to content
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

Don't rely on PATH to find sphinx-build #180

Merged

Conversation

delfick
Copy link
Contributor

@delfick delfick commented Sep 16, 2024

This change makes it so that we don't rely on the correct sphinx-build being on PATH to build the docs.

This is useful when the command that runs sphinx-autobuild runs in a virtualenv that isn't the currently activated virtualenv (i.e. via uv run --extra docs invocation)

@delfick delfick force-pushed the get-sphinx-build-from-current-python branch from 2000b88 to 689f139 Compare September 18, 2024 01:16
This change makes it so that we don't rely on the correct `sphinx-build`
being on PATH to build the docs.

This is useful when the command that runs sphinx-autobuild runs in a
virtualenv that isn't the currently activated virtualenv (i.e. via `uv
run --extra docs` invocation)
@delfick delfick force-pushed the get-sphinx-build-from-current-python branch from 689f139 to 7c37746 Compare September 18, 2024 01:32
Copy link
Member

@AA-Turner AA-Turner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

A

P.S. Please avoid force-pushing in the future; it makes reviewing harder and we squash-merge all PRs.

@AA-Turner AA-Turner merged commit 7b10420 into sphinx-doc:main Sep 18, 2024
6 checks passed
@delfick
Copy link
Contributor Author

delfick commented Sep 18, 2024

thanks :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants