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

CI: Fix GitHub pages #120

Merged
merged 2 commits into from
Oct 15, 2024
Merged

CI: Fix GitHub pages #120

merged 2 commits into from
Oct 15, 2024

Conversation

2xB
Copy link
Member

@2xB 2xB commented Oct 15, 2024

Due to PEP 668, the recommended way to nowadays install Python software via pip is by creating a virtual environment. There are alternatives, but they are not recommended. Since this is required for our CI to work, this pull request adds a simple virtual environment.

@2xB
Copy link
Member Author

2xB commented Oct 15, 2024

This doesn't work in my test, converting to draft...

@2xB 2xB marked this pull request as draft October 15, 2024 14:25
@2xB 2xB marked this pull request as ready for review October 15, 2024 14:51
@2xB
Copy link
Member Author

2xB commented Oct 15, 2024

It works now! There was a dependency issue ( CrossNox/m2r2#68 ) that now got a workaround by installing an older version of a dependency (docutils).

@richeldichel
Copy link
Contributor

Great thanks!

@richeldichel richeldichel merged commit 64fc939 into KATRIN-Experiment:main Oct 15, 2024
4 checks passed
@2xB
Copy link
Member Author

2xB commented Oct 15, 2024

Corresponding PR that would make the workaround obsolete: CrossNox/m2r2#71

@2xB 2xB deleted the 2xB/fixci branch October 15, 2024 15:31
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