https://github.com/python/cpython/commit/61d5e878925abdcb5dbc1de77bc28445f292dacf
commit: 61d5e878925abdcb5dbc1de77bc28445f292dacf
branch: 3.12
author: Sviatoslav Sydorenko (Святослав Сидоренко) <[email protected]>
committer: hugovk <[email protected]>
date: 2024-07-16T14:15:05Z
summary:
[3.12] Fix using `check_source` flags as bool (GH-121848) (#121855)
files:
M .github/workflows/build.yml
M .github/workflows/reusable-windows.yml
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
index cf3e268f0213b9..a3f8ed1e2fe56c 100644
--- a/.github/workflows/build.yml
+++ b/.github/workflows/build.yml
@@ -35,10 +35,30 @@ jobs:
runs-on: ubuntu-latest
timeout-minutes: 10
outputs:
+ # Some of the referenced steps set outputs conditionally and there may be
+ # cases when referencing them evaluates to empty strings. It is nice to
+ # work with proper booleans so they have to be evaluated through JSON
+ # conversion in the expressions. However, empty strings used like that
+ # may trigger all sorts of undefined and hard-to-debug behaviors in
+ # GitHub Actions CI/CD. To help with this, all of the outputs set here
+ # that are meant to be used as boolean flags (and not arbitrary strings),
+ # MUST have fallbacks with default values set. A common pattern would be
+ # to add ` || false` to all such expressions here, in the output
+ # definitions. They can then later be safely used through the following
+ # idiom in job conditionals and other expressions. Here's some examples:
+ #
+ # if: fromJSON(needs.check_source.outputs.run-docs)
+ #
+ # ${{
+ # fromJSON(needs.check_source.outputs.run_tests)
+ # && 'truthy-branch'
+ # || 'falsy-branch'
+ # }}
+ #
run-docs: ${{ steps.docs-changes.outputs.run-docs || false }}
- run_tests: ${{ steps.check.outputs.run_tests }}
- run_hypothesis: ${{ steps.check.outputs.run_hypothesis }}
- config_hash: ${{ steps.config_hash.outputs.hash }}
+ run_tests: ${{ steps.check.outputs.run_tests || false }}
+ run_hypothesis: ${{ steps.check.outputs.run_hypothesis || false }}
+ config_hash: ${{ steps.config_hash.outputs.hash }} # str
steps:
- uses: actions/checkout@v4
- name: Check for source changes
diff --git a/.github/workflows/reusable-windows.yml
b/.github/workflows/reusable-windows.yml
index 41f2c2b0c2cd25..de30c980e22f86 100644
--- a/.github/workflows/reusable-windows.yml
+++ b/.github/workflows/reusable-windows.yml
@@ -2,6 +2,7 @@ on:
workflow_call:
inputs:
free-threading:
+ description: Whether to compile CPython in free-threading mode
required: false
type: boolean
default: false
_______________________________________________
Python-checkins mailing list -- [email protected]
To unsubscribe send an email to [email protected]
https://mail.python.org/mailman3/lists/python-checkins.python.org/
Member address: [email protected]
