FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

gh-111062: Reusable Windows build that supports free-threaded mode as the conditional CI by corona10 · Pull Request #111493 · python/cpython · GitHub

Repository navigation

gh-111062: Reusable Windows build that supports free-threaded mode as the conditional CI - #111493

Merged
corona10 merged 8 commits into
python:mainfrom
corona10:gh-111062-windows
Oct 30, 2023
Merged

corona10 merged 8 commits into
python:mainfrom
corona10:gh-111062-windows

Conversation

corona10 commented Oct 30, 2023 •
edited by bedevere-app Bot
Loading

Copy link
Copy Markdown
Member

ghost commented Oct 30, 2023 •
edited by ghost
Loading

Copy link
Copy Markdown

All commit authors signed the Contributor License Agreement.

Copy link
Copy Markdown
Member Author

co-work with @jota12x


jobs:
build_win_32:
name: 'build and test (x86)'

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

@hugovk I would like to use the name as build and test variations for macOS and Linux too.
It looks more clear when we look at the CI status bar.
But I will do it as a separate PR.

Copy link
Copy Markdown
Member Author

Reopen the PR for testing topic-free-threaded label.

hugovk changed the title gh-111062: Reusable windows build that supports free-threaded mode as the conditional CI gh-111062: Reusable Windows build that supports free-threaded mode as the conditional CI Oct 30, 2023
Co-authored-by: Donghee Na <donghee.na@python.org>
Comment thread .github/workflows/build.yml Outdated
Comment thread .github/workflows/build.yml Outdated
corona10 and others added 3 commits October 30, 2023 23:35
Co-authored-by: Hugo van Kemenade <hugovk@users.noreply.github.com>
corona10 requested a review from hugovk October 30, 2023 14:38

Copy link
Copy Markdown
Member Author

@hugovk For #110764, I merged the PR. Let's do it at the separate PR if there is something what we should do

corona10 merged commit 4ebf2fa into python:main Oct 30, 2023
corona10 deleted the gh-111062-windows branch October 30, 2023 15:30

Copy link
Copy Markdown
Member

On a doc PR, the CI failed because Windows was skipped: https://github.com/python/cpython/actions/runs/6696696148/job/18198167333?pr=111512

It's surprising that once the job is called build_windows, but later build_win_amd64 is allowed to be skipped. Is it a typo?

cc @Eclips4

Copy link
Copy Markdown
Member

Ah, #111521 was proposed to fix the job.

FullteaR pushed a commit to FullteaR/cpython that referenced this pull request Nov 3, 2023
…ode as the conditional CI (python#111493)

* pythongh-111062 Set up free-threaded CI for windows

Co-authored-by: Donghee Na <donghee.na@python.org>

* Apply suggestions from code review

Co-authored-by: Hugo van Kemenade <hugovk@users.noreply.github.com>

* Update

* Update names

* Add files

* Update

* Revert "Update"

This reverts commit 4f17e1a.

* fix

---------

Co-authored-by: juanjose.tenorio <juanjose2611@gmail.com>
Co-authored-by: Hugo van Kemenade <hugovk@users.noreply.github.com>
aisk pushed a commit to aisk/cpython that referenced this pull request Feb 11, 2024
…ode as the conditional CI (python#111493)

* pythongh-111062 Set up free-threaded CI for windows

Co-authored-by: Donghee Na <donghee.na@python.org>

* Apply suggestions from code review

Co-authored-by: Hugo van Kemenade <hugovk@users.noreply.github.com>

* Update

* Update names

* Add files

* Update

* Revert "Update"

This reverts commit 4f17e1a.

* fix

---------

Co-authored-by: juanjose.tenorio <juanjose2611@gmail.com>
Co-authored-by: Hugo van Kemenade <hugovk@users.noreply.github.com>
Glyphack pushed a commit to Glyphack/cpython that referenced this pull request Sep 2, 2024
…ode as the conditional CI (python#111493)

* pythongh-111062 Set up free-threaded CI for windows

Co-authored-by: Donghee Na <donghee.na@python.org>

* Apply suggestions from code review

Co-authored-by: Hugo van Kemenade <hugovk@users.noreply.github.com>

* Update

* Update names

* Add files

* Update

* Revert "Update"

This reverts commit 4f17e1a.

* fix

---------

Co-authored-by: juanjose.tenorio <juanjose2611@gmail.com>
Co-authored-by: Hugo van Kemenade <hugovk@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants


Back | FazBrowse Home | New Git URL