Title: gh-141004: GHA: Run `check-c-api-docs` check on docs-only PRs by StanFromIreland · Pull Request #143573 · python/cpython · GitHub
Open Graph Title: gh-141004: GHA: Run `check-c-api-docs` check on docs-only PRs by StanFromIreland · Pull Request #143573 · python/cpython
X Title: gh-141004: GHA: Run `check-c-api-docs` check on docs-only PRs by StanFromIreland · Pull Request #143573 · python/cpython
Description: As mentioned in #143564 (review) by @ZeroIntensity: One extra thing I would like to do is run the script on docs PRs, since a name being documented while also being in ignored_c_api.txt will cause it to fail. So, if someone merges new documentation without updating ignored_c_api.txt, we'll see blocking failures on all C code PRs until we fix it. I haven't looked too closely at the GHA for change detection, but if you see an easy way to do that, would you mind doing it in this PR? We could keep it in the check-generated-files job, but that would make it quite messy as all the other checks would have to be excluded in the case when only run-docs is true. And, to avoid having to configure, I run the script directly (since it uses PYTHON_FOR_REGEN anyway, there shouldn't be a difference). On this PR, the new job ran in 13 seconds. Issue: gh-141004
Open Graph Description: As mentioned in #143564 (review) by @ZeroIntensity: One extra thing I would like to do is run the script on docs PRs, since a name being documented while also being in ignored_c_api.txt will cause...
X Description: As mentioned in #143564 (review) by @ZeroIntensity: One extra thing I would like to do is run the script on docs PRs, since a name being documented while also being in ignored_c_api.txt will cause...
Opengraph URL: https://github.com/python/cpython/pull/143573
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:6cf0efe2-c2c2-3f0a-6c94-358030866ab1 |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | 86B8:166D49:1373967:1AD625E:696A9242 |
| html-safe-nonce | 6a03d96208653ff4d5cf471fced6cc2b8ab0df0d792210fcf1e0694959a14b67 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiI4NkI4OjE2NkQ0OToxMzczOTY3OjFBRDYyNUU6Njk2QTkyNDIiLCJ2aXNpdG9yX2lkIjoiODkzNjk2MjQwNjQ4MjYxMjgwMiIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9 |
| visitor-hmac | 05d4079785b8524de3ea4de784fad12fb1cfdf5b357614ee60f2e7e0807394e6 |
| hovercard-subject-tag | pull_request:3156780919 |
| github-keyboard-shortcuts | repository,pull-request-list,pull-request-conversation,pull-request-files-changed,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/python/cpython/pull/143573/files |
| twitter:image | https://avatars.githubusercontent.com/u/89152624?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/89152624?s=400&v=4 |
| og:image:alt | As mentioned in #143564 (review) by @ZeroIntensity: One extra thing I would like to do is run the script on docs PRs, since a name being documented while also being in ignored_c_api.txt will cause... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | 4dd496afc954da9c207b2d9fbe86e3074619f565754aa1d9274aec30d9e5b8d7 |
| turbo-cache-control | no-preview |
| diff-view | unified |
| go-import | github.com/python/cpython git https://github.com/python/cpython.git |
| octolytics-dimension-user_id | 1525981 |
| octolytics-dimension-user_login | python |
| octolytics-dimension-repository_id | 81598961 |
| octolytics-dimension-repository_nwo | python/cpython |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 81598961 |
| octolytics-dimension-repository_network_root_nwo | python/cpython |
| turbo-body-classes | logged-out env-production page-responsive full-width |
| disable-turbo | true |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 31496a13b80a2f6ad77d6c617ee2255a3545b539 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width