-
Notifications
You must be signed in to change notification settings - Fork 195
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: add support for tokenless v3 #1410
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1410 +/- ##
==========================================
- Coverage 91.66% 90.35% -1.32%
==========================================
Files 4 4
Lines 312 311 -1
Branches 85 84 -1
==========================================
- Hits 286 281 -5
- Misses 26 30 +4
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
src/buildExec.ts
Outdated
@@ -31,7 +31,7 @@ const getGitService = (): string => { | |||
|
|||
const isFork = (): boolean => { | |||
if ( | |||
`${context.eventName}` !== 'pull_request' || | |||
`${context.eventName}` !== 'pull_request' && |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good catch
9a328fc
to
b7e79c8
Compare
@@ -47,6 +47,7 @@ const isPullRequestFromFork = (): boolean => { | |||
const getToken = async (): Promise<string> => { | |||
if (isPullRequestFromFork()) { | |||
core.info('==> Fork detected, tokenless uploading used'); | |||
process.env['TOKENLESS'] = context.payload.pull_request.head.label; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you in this PR description link to the CLI change that is consuming this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just need some documentation to how this will be consumed
@joseph-sentry you will need to run |
b7e79c8
to
a193b6d
Compare
a193b6d
to
6c9378e
Compare
The idea is that we will detect that if this action is being run from a PR and that PR is from a fork to an upstream repo then we should upload using tokenless, so we will set the TOKENLESS env var so that we can pass the necessary information for the CLI to upload using tokenless
Necessary for: codecov/codecov-cli#434