Title: Make matrix run all cases of the python-deps tests by robertbrignull · Pull Request #482 · github/codeql-action · GitHub
Open Graph Title: Make matrix run all cases of the python-deps tests by robertbrignull · Pull Request #482 · github/codeql-action
X Title: Make matrix run all cases of the python-deps tests by robertbrignull · Pull Request #482 · github/codeql-action
Description: The python-dep tests didn't appear to be running on the cases that were intended. I don't think the use of include was correct, or possibly there might be a bug in how it's being parsed / handled. Either way I'm pretty sure it's not how that feature was intended to be used. I'm going off off the information at https://docs.github.com/en/actions/reference/workflow-syntax-for-github-actions#example-including-additional-values-into-combinations This PR removes the use of include in favour of a more standard matrix, though this did require splitting one job up into two. If this works then it means we'll now be running 26 jobs. So another question is is it necessary to run this many variations? Merge / deployment checklist Confirm this change is backwards compatible with existing workflows. Confirm the readme has been updated if necessary.
Open Graph Description: The python-dep tests didn't appear to be running on the cases that were intended. I don't think the use of include was correct, or possibly there might be a bug in how it's being parsed...
X Description: The python-dep tests didn't appear to be running on the cases that were intended. I don't think the use of include was correct, or possibly there might be a bug in how it's ...
Opengraph URL: https://github.com/github/codeql-action/pull/482
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:6b67b783-8ee4-a6da-3f49-658e0d4a6ece |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | 9C60:3819A0:717FEF:9BC897:6A55A697 |
| html-safe-nonce | ab04983d1006059064b3f4e5687e80166417ff12e318ca0aebb01b953e2b6e48 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiI5QzYwOjM4MTlBMDo3MTdGRUY6OUJDODk3OjZBNTVBNjk3IiwidmlzaXRvcl9pZCI6Ijc2MjIzOTA5MDQyMzkwMDczODMiLCJyZWdpb25fZWRnZSI6ImlhZCIsInJlZ2lvbl9yZW5kZXIiOiJpYWQifQ== |
| visitor-hmac | 1337e815a3359457ad05ce8695c6c6d7c86a814db149c541f8c9998ed17d0064 |
| hovercard-subject-tag | pull_request:632455515 |
| 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/github/codeql-action/pull/482/files |
| twitter:image | https://avatars.githubusercontent.com/u/3749000?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/3749000?s=400&v=4 |
| og:image:alt | The python-dep tests didn't appear to be running on the cases that were intended. I don't think the use of include was correct, or possibly there might be a bug in how it's being parsed... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | b5665c84831ed9ac4fb79519c16c9c5580ba8092fb8bb6e3e72972ec7197348e |
| turbo-cache-control | no-preview |
| diff-view | unified |
| go-import | github.com/github/codeql-action git https://github.com/github/codeql-action.git |
| octolytics-dimension-user_id | 9919 |
| octolytics-dimension-user_login | github |
| octolytics-dimension-repository_id | 259445878 |
| octolytics-dimension-repository_nwo | github/codeql-action |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 259445878 |
| octolytics-dimension-repository_network_root_nwo | github/codeql-action |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | true |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 653fccf1e27344851c0d3de3fa6faa526197f072 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width