vscode-sqltools icon indicating copy to clipboard operation
vscode-sqltools copied to clipboard

Build failures - has CODECOV_TOKEN expired?

Open gjsjohnmurray opened this issue 3 years ago • 3 comments

See https://github.com/mtxr/vscode-sqltools/runs/7577287610

gjsjohnmurray avatar Jul 29 '22 10:07 gjsjohnmurray

Still seems to be broken. See https://github.com/mtxr/vscode-sqltools/actions/runs/2776548495

gjsjohnmurray avatar Aug 01 '22 17:08 gjsjohnmurray

https://github.com/mtxr/vscode-sqltools/actions/workflows/test-formatter.yml @gjsjohnmurray newer builds are working fine apparently.

I see no issue when I go to the actions page. the link you shared is from a PR that had a outdated branch. the formatter token was renamed to CODECOV_TOKEN_FORMATTER, so old PRs might still fail :/. newer will work just fine

mtxr avatar Aug 03 '22 00:08 mtxr

@mtxr they've only been working because I created https://github.com/mtxr/vscode-sqltools/pull/929 to comment out the upload to codecov again, I got @george-james-software to approve it. Then I merged it to dev, then merged dev into a dependabot PR, approved it, used gh api to manually set the build (10.x) status check to success, then merged the PR. Repeated that cycle lots of times until all the dependabot PRs were merged.

gjsjohnmurray avatar Aug 04 '22 14:08 gjsjohnmurray

Hmm, still not working for Formatter Test & Build actions triggered by my pushes.

For example:

https://github.com/mtxr/vscode-sqltools/runs/7949447061?check_suite_focus=true

Details:

image

Compare with this one triggered by a push by @mtxr:

https://github.com/mtxr/vscode-sqltools/runs/7937133670?check_suite_focus=true

Details which I think show this run has more permissions:

image

gjsjohnmurray avatar Aug 22 '22 10:08 gjsjohnmurray

@mtxr maybe the test-formatter.yml action should only do its codecov/[email protected] step on pull_request and not on push to the dev branch, because in the latter case it seems the action job has fewer permissions (perhaps because it is operating on files that may have been pushed from a fork so shouldn't yet be trusted).

gjsjohnmurray avatar Aug 22 '22 10:08 gjsjohnmurray