Skip to content

Upgrade Python version in Windows CI#571

Merged
AdamGleave merged 1 commit into
masterfrom
upgrade-windows-ci
Sep 29, 2022
Merged

Upgrade Python version in Windows CI#571
AdamGleave merged 1 commit into
masterfrom
upgrade-windows-ci

Conversation

@Rocamonde

Copy link
Copy Markdown
Member

Description

Windows is currently failing CI in some PRs (e.g. https://app.circleci.com/pipelines/github/HumanCompatibleAI/imitation/2721/workflows/015d1fb1-21f8-4245-aeae-85a8f0524c24/jobs/8509/steps, #537) due to a bug in an old version of CPython (python/cpython#83396). This was fixed in python/cpython#17827, but since chocolatey is installing python 3.8.0., this (and other CPython bugs) are still present.

This PR upgrades python on windows to the most recent version available on chocolatey (3.8.10). At the time of writing, the most recent python version is actually 3.8.14.

@codecov

codecov Bot commented Sep 29, 2022

Copy link
Copy Markdown

Codecov Report

Merging #571 (7293915) into master (fe6e5f5) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #571   +/-   ##
=======================================
  Coverage   97.23%   97.23%           
=======================================
  Files          88       88           
  Lines        8028     8028           
=======================================
  Hits         7806     7806           
  Misses        222      222           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@AdamGleave AdamGleave left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM. Weird choco doesn't just default to installing the most recent version 🤷

@AdamGleave AdamGleave merged commit 0f0c755 into master Sep 29, 2022
@AdamGleave AdamGleave deleted the upgrade-windows-ci branch September 29, 2022 17:34
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