Title: Avoid sending status reports on enterprise by robertbrignull · Pull Request #128 · github/codeql-action · GitHub
Open Graph Title: Avoid sending status reports on enterprise by robertbrignull · Pull Request #128 · github/codeql-action
X Title: Avoid sending status reports on enterprise by robertbrignull · Pull Request #128 · github/codeql-action
Description: This avoids us sending status reports when running on enterprise. The endpoint currently doesn't do anything that would be useful on enterprise. This will probably conflict a little bit with #117 but I'll fix that up when it happens. The check and the point where we revert is currently just before we actually send the status report, but notably this is after we've constructed the report, so there is a little bit of wasted effort. I think this is acceptable for the first attempt, though there's clearly room for improvement to avoid constructing the report while also keeping the centralised check and avoiding having to add checks to every call. Things that need to be checked and tested before merging: Status reports are still sent as normal on github.com Status reports are not sent on enterprise Merge / deployment checklist Confirm this change is backwards compatible with existing workflows. Confirm the readme has been updated if necessary.
Open Graph Description: This avoids us sending status reports when running on enterprise. The endpoint currently doesn't do anything that would be useful on enterprise. This will probably conflict a little bit with #1...
X Description: This avoids us sending status reports when running on enterprise. The endpoint currently doesn't do anything that would be useful on enterprise. This will probably conflict a little bit wit...
Opengraph URL: https://github.com/github/codeql-action/pull/128
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:34f3ef74-ba7e-c4d3-e6b9-2e1c81951796 |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | 8FCC:15921E:258F22:353802:6A5724D0 |
| html-safe-nonce | f69aff05266b19c72c24ac02df1662363307c789c3d0235d528769751f28efa2 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiI4RkNDOjE1OTIxRToyNThGMjI6MzUzODAyOjZBNTcyNEQwIiwidmlzaXRvcl9pZCI6IjIwMDMyOTA4NDMwNTkyNjY3NjgiLCJyZWdpb25fZWRnZSI6ImlhZCIsInJlZ2lvbl9yZW5kZXIiOiJpYWQifQ== |
| visitor-hmac | e455a2ee422bf948b6b29221b3b9dc0b7d129ed82c4c9dae255dfe174c3a9352 |
| hovercard-subject-tag | pull_request:459236572 |
| 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/128/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 | This avoids us sending status reports when running on enterprise. The endpoint currently doesn't do anything that would be useful on enterprise. This will probably conflict a little bit with #1... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | 4e7a7296a3830877cf21a6ad2a972c9e618a48915e03966cf0c53eb08e5aad98 |
| 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 | 2576d1f0198cf1588faf2edf27f1ed120903be10 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width