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: ensure Python >=3 is installed when building the documentation #441

Merged
merged 1 commit into from
Dec 12, 2023

Conversation

pnmadelaine
Copy link
Contributor

The workflow to build the documentation is currently flaky.
Pip sometimes fails with the following error:

DEPRECATION: Python 2.7 reached the end of its life on January 1st, 2020. Please upgrade your Python as Python 2.7 is no longer maintained. pip 21.0 will drop support for Python 2.7 in January 2021. More details about Python 2 support in pip can be found at https://pip.pypa.io/en/latest/development/release-process/#python-2-support pip 21.0 will remove support for this functionality.
ERROR: Could not find a version that satisfies the requirement myst-parser (from -r requirements.txt (line 1)) (from versions: none)
ERROR: No matching distribution found for myst-parser (from -r requirements.txt (line 1))

Thus it seems that the macos runners sometimes come with Python 2 installed which causes pip to fail.
What is weird is that this only happens sometimes, and a re-run of the job usually fixes the issue.

This PR uses the setup-python github action to ensure that a recent version of Python is installed.

@pnmadelaine pnmadelaine requested a review from a team as a code owner December 11, 2023 13:51
@cla-bot cla-bot bot added the cla-signed label Dec 11, 2023
@coveralls
Copy link

Pull Request Test Coverage Report for Build 7168415228

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 54.055%

Totals Coverage Status
Change from base Build 7115336754: 0.0%
Covered Lines: 27829
Relevant Lines: 51483

💛 - Coveralls

Copy link
Member

@franziskuskiefer franziskuskiefer left a comment

Choose a reason for hiding this comment

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

Thanks. Let's try this.

@franziskuskiefer franziskuskiefer merged commit 2d91914 into main Dec 12, 2023
40 checks passed
@franziskuskiefer franziskuskiefer deleted the pnmadelaine-fix-doc branch December 12, 2023 16:12
protz pushed a commit that referenced this pull request Apr 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants