Skip to content

Import version from installed package if the file doesnt exist#644

Merged
rahulporuri merged 2 commits into
masterfrom
fix/build-docs
Apr 12, 2021
Merged

Import version from installed package if the file doesnt exist#644
rahulporuri merged 2 commits into
masterfrom
fix/build-docs

Conversation

@rahulporuri
Copy link
Copy Markdown
Contributor

If the package is installed in a non-editable way, the _version file won't exist where we expect it to. This PR updates the sphinx configuration to get the version information from the package directly if the file doesn't exist.

If the package is installed in a non-editable way, the file won't exist
in the location we think it exists - so we need to fall back to
importing the version information directly from the installed package

	modified:   docs/source/conf.py
Copy link
Copy Markdown
Contributor

@aaronayres35 aaronayres35 left a comment

Choose a reason for hiding this comment

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

LGTM

just one comment about using chaco instead of enable.

Comment thread docs/source/conf.py Outdated
Co-authored-by: aaronayres35 <36972686+aaronayres35@users.noreply.github.com>
@rahulporuri
Copy link
Copy Markdown
Contributor Author

rahulporuri commented Apr 12, 2021

sigh. thanks for catching that silly silly mistake

@rahulporuri rahulporuri merged commit f7cdff7 into master Apr 12, 2021
@rahulporuri rahulporuri deleted the fix/build-docs branch April 12, 2021 14:36
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