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
Pin npe2 version to match installed one #175
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me with one confusing piece.
Co-authored-by: Ashley Anderson <aganders3@gmail.com>
Does this affect the |
if not parse(npe2_version).is_devrelease: | ||
check_call(f"git checkout tags/v{npe2_version}".split(), cwd=NPE) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This check disable checkout if development version of npe2 is installed in workflow. So it is enough to add pip install git+https://github.com/napari/npe2
to the workflow to have dev version of docs.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems good to me. I was able to complete the build. However in the environment on gitpod I was not able to complete the build with PyQt5. I had to use Pyside or otherwise I would get an error for a visual draw. I will open an issue for this.
I will merge it not to restore working napari repo tests |
Description
The napari/npe2#294 breaks the docs building as we use the most recent npe2 repository, not matching npe2 version. In this PR I add
git checkout call
to have the npe2 repository cloned in a version matching the installed npe2 version.Type of change
References
Final checklist: