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

Fix CI #538

Merged
merged 7 commits into from
Apr 21, 2023
Merged

Fix CI #538

merged 7 commits into from
Apr 21, 2023

Conversation

thet
Copy link
Member

@thet thet commented Apr 21, 2023

See individual commits.

@jensens @fredvd @mauritsvanrees @petschki added you all as reviewers because you were wether involved in this or should know about.

fredvd and others added 2 commits April 21, 2023 23:29
This is deprecated in pip 23 which we switched to recently.  --install option was passed to setuptools, but pip is now using wheel install only from what I understand. --pre signals using pre-releases when available, and it's also a pip option. Lets pass it directly to pip and see what happens.

More info:
pypa/pip#11358
https://discuss.python.org/t/passing-command-line-arguments-to-pip-install-after-install-options-deprecation/22981/6
Disable source checkout of sphinxcontrib_robotframework due to an error
with "--install-option='--pre'" in requirement files and recent pip
versions.

As soon this is fixed upstream we can re-enable it.

Ref:

- mxstack/mxdev#30
- collective/sphinxcontrib-robotframework#6
…ixed."

With the pip==23.0 fixture we can again checkout
sphinxcontrib_robotframework.

This reverts commit 91012cf.
@thet thet merged commit 455c6ee into master Apr 21, 2023
@thet thet deleted the fix-ci branch April 21, 2023 23:11
mauritsvanrees added a commit that referenced this pull request Apr 22, 2023
Taken over from #538
thet pushed a commit that referenced this pull request Apr 22, 2023
Taken over from #538
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