You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Check if generated files are up-to-date in Github Actions #86378
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
assignee=Noneclosed_at=<Date2020-11-24.13:34:14.078>created_at=<Date2020-10-30.17:22:37.069>labels= ['build', '3.8', '3.9', '3.10']
title='Check if generated files are up-to-date in Github Actions'updated_at=<Date2021-03-02.20:39:17.985>user='https://github.com/FFY00'
script:
# Using the built Python as patchcheck.py is built around the idea of using
# a checkout-build of CPython to know things like what base branch the changes
# should be compared against.
# Only run on Linux as the check only needs to be run once.
if [[ "$TRAVIS_OS_NAME" == "linux" ]]; then ./python Tools/scripts/patchcheck.py --travis $TRAVIS_PULL_REQUEST; fi
script: # Using the built Python as patchcheck.py is built around the idea of using # a checkout-build of CPython to know things like what base branch the changes # should be compared against. # Only run on Linux as the check only needs to be run once.
if [[ "$TRAVIS_OS_NAME" == "linux" ]]; then ./python Tools/scripts/patchcheck.py --travis $TRAVIS_PULL_REQUEST; fi
@vstinner Did this happen? I currently don't see a patchcheck job on my PRs, and think it should be added.
Activity
FFY00 commentedon Oct 30, 2020
python/core-workflow#380
vstinner commentedon Nov 20, 2020
New changeset d20b7ed by Filipe Laíns in branch 'master':
bpo-42212: Check if generated files are up-to-date in GitHub Actions (GH-23042)
d20b7ed
vstinner commentedon Nov 20, 2020
Let's see how it goes on PRs for a few days, and later we can decide if it should be backported to 3.8 and 3.9 branches.
vstinner commentedon Nov 20, 2020
While we are here... it might interesting to also run patchcheck.py on PRs using GitHub Actions:
https://mail.python.org/archives/list/python-dev@python.org/message/DKLD2VLHZ3F54WTKZ7JHKWBMOZ4N5TK7/
.travis.yml uses:
script:
# Using the built Python as patchcheck.py is built around the idea of using
# a checkout-build of CPython to know things like what base branch the changes
# should be compared against.
# Only run on Linux as the check only needs to be run once.
vstinner commentedon Nov 24, 2020
New changeset ac7d016 by Victor Stinner in branch 'master':
bpo-42212: smelly.py also checks the dynamic library (GH-23423)
ac7d016
miss-islington commentedon Nov 24, 2020
New changeset 15d42d7 by Miss Islington (bot) in branch '3.9':
bpo-42212: Check if generated files are up-to-date in GitHub Actions (GH-23042)
15d42d7
miss-islington commentedon Nov 24, 2020
New changeset b55a276 by Miss Islington (bot) in branch '3.8':
bpo-42212: Check if generated files are up-to-date in GitHub Actions (GH-23042)
b55a276
vstinner commentedon Nov 24, 2020
Thanks Filipe Laíns! I merged your PR on 3.8, 3.9 and master branches.
itaisteinherz commentedon May 10, 2022
@vstinner Did this happen? I currently don't see a patchcheck job on my PRs, and think it should be added.
vstinner commentedon May 10, 2022
I don't know. If it's missing, you can propose a PR to add the check.