Skip to content

MAINT, CI: fix doc build#9648

Merged
drammock merged 1 commit intomne-tools:mainfrom
drammock:fix-doc-build
Aug 6, 2021
Merged

MAINT, CI: fix doc build#9648
drammock merged 1 commit intomne-tools:mainfrom
drammock:fix-doc-build

Conversation

@drammock
Copy link
Copy Markdown
Member

@drammock drammock commented Aug 5, 2021

#9634 broke documentation builds because it tried to parse setup.py to get the required python version (setup.py is not distributed with the package, so it can only be used that way in git-clone-based installations).

This uses either importlib (py 3.8+) or pkg_resources (py3.7) instead.

@drammock
Copy link
Copy Markdown
Member Author

drammock commented Aug 5, 2021

note: I'm expecting the CIs to be green here (we should wait and see), but the real test will be in the deploy step after this is merged.

@drammock drammock added the backport-candidate on-merge: backport to maint/1.12 label Aug 5, 2021
@drammock
Copy link
Copy Markdown
Member Author

drammock commented Aug 5, 2021

rendered doc looks okay:

(look at the first paragraph of each page, it says correctly "Python 3.7"; the 3.7 part is rendered from a variable)

@drammock drammock merged commit c556894 into mne-tools:main Aug 6, 2021
@drammock drammock deleted the fix-doc-build branch August 6, 2021 14:15
drammock added a commit that referenced this pull request Aug 6, 2021
@drammock drammock added backported and removed backport-candidate on-merge: backport to maint/1.12 labels Aug 13, 2021
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.

1 participant