Title: Install Python tools on self-hosted runners by edoardopirovano · Pull Request #618 · github/codeql-action · GitHub
Open Graph Title: Install Python tools on self-hosted runners by edoardopirovano · Pull Request #618 · github/codeql-action
X Title: Install Python tools on self-hosted runners by edoardopirovano · Pull Request #618 · github/codeql-action
Description: This PR changes the init step to also install the Python packages that the Action requires when running on self-hosted runners, as we already do on GitHub ones. The rationale given in #155 for not doing this was that "we don't want our cli to be messing around with things in the system". On the one hand, this seems reasonable. However, I think we should try to make a best attempt to succeed with our analysis and in my opinion this means installing tools we require. Notably, we are already messing around with things that are installed since the later step will install an arbitrary number of packages that are dependencies of the code so that we can analyse it. If we are happy to do this, I see no reason why we shouldn't also be happy to install the four Python packages that we need. This PR was prompted by #615, where the user's workflow failed due to this (and one other issue). Merge / deployment checklist Confirm this change is backwards compatible with existing workflows. Confirm the readme has been updated if necessary. Confirm the changelog has been updated if necessary.
Open Graph Description: This PR changes the init step to also install the Python packages that the Action requires when running on self-hosted runners, as we already do on GitHub ones. The rationale given in #155 for not ...
X Description: This PR changes the init step to also install the Python packages that the Action requires when running on self-hosted runners, as we already do on GitHub ones. The rationale given in #155 for not ...
Opengraph URL: https://github.com/github/codeql-action/pull/618
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:6cbb484d-1d62-b492-f3a9-31b730d798d5 |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | EDD0:17402E:4CA579:68CD20:6A540AD7 |
| html-safe-nonce | 1183c7928d72bbaff6c4df563d732bf694b7d0453bb5176e4e0d36b8efe07b4d |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJFREQwOjE3NDAyRTo0Q0E1Nzk6NjhDRDIwOjZBNTQwQUQ3IiwidmlzaXRvcl9pZCI6IjIzMTgyMTkyOTg1NjM0MjcwMzEiLCJyZWdpb25fZWRnZSI6ImlhZCIsInJlZ2lvbl9yZW5kZXIiOiJpYWQifQ== |
| visitor-hmac | b60ff93336c3bf6ee43ea37d8f99431701a23fafb3e8375db883f3078d8a9d0b |
| hovercard-subject-tag | pull_request:688589735 |
| 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/618/files |
| twitter:image | https://avatars.githubusercontent.com/u/6748066?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/6748066?s=400&v=4 |
| og:image:alt | This PR changes the init step to also install the Python packages that the Action requires when running on self-hosted runners, as we already do on GitHub ones. The rationale given in #155 for not ... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | b9a586c06a05a7a86fc7e3f4dbd03e42f6869085879aa184aa6369456dbd50fb |
| 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 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 | 07a982c1d40157c619b364352b704c3ce66bb332 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width