Title: 🐛 fix bug where build/smoke test checkout action usage checks out head of main branch by ryandens · Pull Request #342 · hypertrace/javaagent · GitHub
Open Graph Title: 🐛 fix bug where build/smoke test checkout action usage checks out head of main branch by ryandens · Pull Request #342 · hypertrace/javaagent
X Title: 🐛 fix bug where build/smoke test checkout action usage checks out head of main branch by ryandens · Pull Request #342 · hypertrace/javaagent
Description: Description For some reason, actions that use the pull_request_target attribute causes the deefault checkout destination of the checkout action to be whatever is currently the head of origin/main. This can allow build/test errors from PRs make it into the main branch before failing (but not allowing them to be released). An example PR is #337. Note, the GHA all passed on commit e9d665c. However, note the checkout action step for that PR here. It checks out the commit from what was currently the head of origin/main (ref 4a920915c55e3d0e86806482bec736554315100b) and caused the PR to pass all builds, only to have the main build fail once the PR was merged. This resulted in us needing to quickly patch in PR #338. Testing Please describe the tests that you ran to verify your changes. Please summarize what did you test and what needs to be tested e.g. deployed and tested helm chart locally. Checklist: My changes generate no new warnings I have added tests that prove my fix is effective or that my feature works Any dependent changes have been merged and published in downstream modules Documentation Make sure that you have documented corresponding changes in this repository or hypertrace docs repo if required.
Open Graph Description: Description For some reason, actions that use the pull_request_target attribute causes the deefault checkout destination of the checkout action to be whatever is currently the head of origin/main. ...
X Description: Description For some reason, actions that use the pull_request_target attribute causes the deefault checkout destination of the checkout action to be whatever is currently the head of origin/main. ...
Opengraph URL: https://github.com/hypertrace/javaagent/pull/342
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/files(.:format) |
| route-controller | pull_requests |
| route-action | files |
| fetch-nonce | v2:12b532da-4cbe-d6e4-b92a-38bf4834de8b |
| current-catalog-service-hash | ae870bc5e265a340912cde392f23dad3671a0a881730ffdadd82f2f57d81641b |
| request-id | CFEA:11EE4:26CABF:3784B1:6A4E6796 |
| html-safe-nonce | 36627b9c17657f25d34eb7e9d7ffea58d04463187dc987ebd964172b8bf2c157 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJDRkVBOjExRUU0OjI2Q0FCRjozNzg0QjE6NkE0RTY3OTYiLCJ2aXNpdG9yX2lkIjoiMzUzNDE5MTEzMzUyODU4MjAzOCIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9 |
| visitor-hmac | b90e1e866ff26f4dc53f6d157ba88be7f6dc46f434b44b89b7d852c704997d68 |
| hovercard-subject-tag | pull_request:695911939 |
| 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/hypertrace/javaagent/pull/342/files |
| twitter:image | https://avatars.githubusercontent.com/u/8192732?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/8192732?s=400&v=4 |
| og:image:alt | Description For some reason, actions that use the pull_request_target attribute causes the deefault checkout destination of the checkout action to be whatever is currently the head of origin/main. ... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | 41b6ab3ba6d20a71766ac245b5a4a94c6fc672a9cd4da7d44c1b33ab8bf6a21c |
| turbo-cache-control | no-preview |
| diff-view | unified |
| go-import | github.com/hypertrace/javaagent git https://github.com/hypertrace/javaagent.git |
| octolytics-dimension-user_id | 65374698 |
| octolytics-dimension-user_login | hypertrace |
| octolytics-dimension-repository_id | 302314733 |
| octolytics-dimension-repository_nwo | hypertrace/javaagent |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 302314733 |
| octolytics-dimension-repository_network_root_nwo | hypertrace/javaagent |
| 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 | e6a744804e8e70f97b4d5a18a94dcc63db22f97a |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width