Title: Remove some shared env by robertbrignull · Pull Request #140 · github/codeql-action · GitHub
Open Graph Title: Remove some shared env by robertbrignull · Pull Request #140 · github/codeql-action
X Title: Remove some shared env by robertbrignull · Pull Request #140 · github/codeql-action
Description: This PR aims to remove a few elements from shared-environment.ts. Most of this is pretty easy because the variable is only used from one place, or the data is readily available in the config already. This doesn't remove ODASA_TRACER_CONFIGURATION or CODEQL_WORKFLOW_STARTED_AT because those are unfortunately going to be slightly more tricky. I think the only bit that needs explaining is the removal of should_abort. I claim this is safe to do because it's functionality of checking that the init action has already run is covered by the call to getConfig in the autobuild and analyze actions. In the init and upload-sarif actions, all it was doing was validating that the github ref env var was populated, which doesn't seem worth it to keep the function around. Merge / deployment checklist Confirm this change is backwards compatible with existing workflows. Confirm the readme has been updated if necessary.
Open Graph Description: This PR aims to remove a few elements from shared-environment.ts. Most of this is pretty easy because the variable is only used from one place, or the data is readily available in the config alread...
X Description: This PR aims to remove a few elements from shared-environment.ts. Most of this is pretty easy because the variable is only used from one place, or the data is readily available in the config alread...
Opengraph URL: https://github.com/github/codeql-action/pull/140
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:4e5cee3e-9054-dd89-8cd4-54e7e9d08f51 |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | 88C0:1748BD:45AA833:624592A:6A56E693 |
| html-safe-nonce | b41865401ad87b3d2e462ef18a9d52bb2670dc41b81d241ae5e0d070ab0423ba |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiI4OEMwOjE3NDhCRDo0NUFBODMzOjYyNDU5MkE6NkE1NkU2OTMiLCJ2aXNpdG9yX2lkIjoiMjg4MDQ0ODI5ODc3NDYxOTc5NSIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9 |
| visitor-hmac | 4149cfba1e0e30b542305651f4ad2231c8f11bcb5752a99e8f2f52df58d69dba |
| hovercard-subject-tag | pull_request:464697411 |
| 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/140/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 PR aims to remove a few elements from shared-environment.ts. Most of this is pretty easy because the variable is only used from one place, or the data is readily available in the config alread... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | 5220a278731283038daf0ed00b41f06ef7ff9539020ecc806c5c5a0fb6bda25a |
| 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 | 8684ae1a81cd3e4ef219b241179982bbda1a5b1f |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width