You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Jan 10, 2023. It is now read-only.
This means that when you run codecov to perform the upload it is returning CODEBUILD_SOURCE_VERSION as the PR number, which throws an error: pr must match pattern ^(\d+|false|null|undefined|true)$
My proposed solution is to just check against CODEBUILD_SOURCE_VERSION and validate that it starts with pr/, then return the parsed-out number. If the validation fails, return undefined.
The text was updated successfully, but these errors were encountered:
The current iteration of the Codebuild config tool resolves the PR field as follows:
Here are what those environment variables look like when you trigger a build via a PR:
...and here are what they look like when triggered from a push/merge: (This is the exact same commit from the PR)
This means that when you run
codecov
to perform the upload it is returningCODEBUILD_SOURCE_VERSION
as the PR number, which throws an error:pr must match pattern ^(\d+|false|null|undefined|true)$
My proposed solution is to just check against
CODEBUILD_SOURCE_VERSION
and validate that it starts withpr/
, then return the parsed-out number. If the validation fails, return undefined.The text was updated successfully, but these errors were encountered: