Skip to content

Conversation

@StanFromIreland
Copy link
Member

@StanFromIreland StanFromIreland commented Jan 8, 2026

@StanFromIreland
Copy link
Member Author

@hugovk maybe you could please take a look?

Copy link
Member

@ZeroIntensity ZeroIntensity left a comment

Choose a reason for hiding this comment

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

Thanks, I'm fine with this. Since it's right before the weekend, I'll give others a few more days to take a look before merging.

Comment on lines +26 to +28
Path(".github/CODEOWNERS"),
Path(".pre-commit-config.yaml"),
Path(".ruff.toml"),
Copy link
Member

Choose a reason for hiding this comment

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

It might be worth keeping CONFIGURATION_FILE_NAMES and just flattening it here, but I'm not going to block the PR on it.

@hugovk hugovk added needs backport to 3.13 bugs and security fixes needs backport to 3.14 bugs and security fixes labels Jan 9, 2026
@hugovk
Copy link
Member

hugovk commented Jan 10, 2026

Idle thought, we could write:

if not (doc_file or file in RUN_TESTS_IGNORE):

As:

if not doc_file and file not in RUN_TESTS_IGNORE:

I think that's a little clearer?


An unrelated condition that could be made clearer in this file is:

        if not has_platform_specific_change or not platforms_changed:
            a
        else:
            b

Is the same as:

        if not (has_platform_specific_change and platforms_changed):
            a
        else:
            b

And flip to put the positive first:

        if has_platform_specific_change and platforms_changed:
            b
        else:
            a

@ZeroIntensity ZeroIntensity merged commit 1176fac into python:main Jan 13, 2026
53 checks passed
@miss-islington-app
Copy link

Thanks @StanFromIreland for the PR, and @ZeroIntensity for merging it 🌮🎉.. I'm working now to backport this PR to: 3.13, 3.14.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jan 13, 2026
…cs/ignored_c_api.txt` is changed (pythonGH-143583)

(cherry picked from commit 1176facbf21388ef29276ec55a95a66423f61191)

Co-authored-by: Stan Ulbrych <[email protected]>
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jan 13, 2026
…cs/ignored_c_api.txt` is changed (pythonGH-143583)

(cherry picked from commit 1176facbf21388ef29276ec55a95a66423f61191)

Co-authored-by: Stan Ulbrych <[email protected]>
@bedevere-app
Copy link

bedevere-app bot commented Jan 13, 2026

GH-143793 is a backport of this pull request to the 3.14 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.14 bugs and security fixes label Jan 13, 2026
@bedevere-app
Copy link

bedevere-app bot commented Jan 13, 2026

GH-143794 is a backport of this pull request to the 3.13 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.13 bugs and security fixes label Jan 13, 2026
@StanFromIreland StanFromIreland deleted the ignore-capi-tests branch January 13, 2026 13:32
ZeroIntensity pushed a commit that referenced this pull request Jan 13, 2026
…ocs/ignored_c_api.txt` is changed (GH-143583) (GH-143794)

gh-141004: Don't trigger `run-tests` when `Tools/check-c-api-docs/ignored_c_api.txt` is changed (GH-143583)
(cherry picked from commit 1176fac)

Co-authored-by: Stan Ulbrych <[email protected]>
ZeroIntensity pushed a commit that referenced this pull request Jan 13, 2026
…ocs/ignored_c_api.txt` is changed (GH-143583) (GH-143793)

gh-141004: Don't trigger `run-tests` when `Tools/check-c-api-docs/ignored_c_api.txt` is changed (GH-143583)
(cherry picked from commit 1176fac)

Co-authored-by: Stan Ulbrych <[email protected]>
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.

3 participants