Skip to content
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

v4: New major version tagged prematurely #1089

Closed
3flex opened this issue Sep 15, 2023 · 6 comments
Closed

v4: New major version tagged prematurely #1089

3flex opened this issue Sep 15, 2023 · 6 comments
Assignees

Comments

@3flex
Copy link

3flex commented Sep 15, 2023

I realise it's probably too late, but FYI using the v4 tag while the new version is in beta causes tools like Renovate (and maybe dependabot?) to automatically open PRs to update to the new version, e.g. detekt/detekt#6490

While it's in beta this is unexpected. The v4 tag should be deleted, and a v4-beta tag used instead.

From https://docs.github.com/en/actions/creating-actions/about-custom-actions#using-tags-for-release-management:

Major versions can be initially released with a beta tag to indicate their status, for example, v2-beta. The -beta tag can then be removed when ready.

@thomasrockhu-codecov
Copy link
Contributor

Sorry about this @3flex, this has been removed and replaced with v4-beta

nijel added a commit to WeblateOrg/meta that referenced this issue Sep 15, 2023
TonyRL added a commit to DIYgod/RSSHub that referenced this issue Sep 15, 2023
@ssbarnea
Copy link

Was anyone able to make the v4 beta version to succeed uploading? I tried and it keeps failing to validate the token, even if I put put the token inside an environment, configured the pipeline to use that environment and passed it, see https://github.com/ansible/ansible-lint/pull/3764/files

Still, pipelines fail complaining about missing token: https://github.com/ansible/ansible-lint/actions/runs/6313000590/job/17140193383 --- when clearly the token was passed.

[8](https://github.com/ansible/ansible-lint/actions/runs/6313000590/job/17140193383#step:15:49)
Error: Codecov: Failed to properly create commit: The process '/home/runner/work/_actions/codecov/codecov-action/v4.0.0-beta.2/dist/codecov' failed with exit code 1

okuramasafumi added a commit to okuramasafumi/alba that referenced this issue Sep 26, 2023
@thomasrockhu-codecov thomasrockhu-codecov self-assigned this Sep 28, 2023
@thomasrockhu-codecov
Copy link
Contributor

@ssbarnea I'll handle this in your issue here

kevinoid added a commit to kevinoid/nodecat that referenced this issue Oct 1, 2023
This reverts commit 7f6eac9.

The tagging of codecov/codecov-action@v4 was premature:
codecov/codecov-action#1089
Wait until v4 is out of beta to upgrade.

Signed-off-by: Kevin Locke <kevin@kevinlocke.name>
kevinoid added a commit to kevinoid/openapi-transformer-base that referenced this issue Oct 2, 2023
This reverts commit 85284b2.

The tagging of codecov/codecov-action@v4 was premature:
codecov/codecov-action#1089
Wait until v4 is out of beta to upgrade.

Signed-off-by: Kevin Locke <kevin@kevinlocke.name>
kevinoid added a commit to kevinoid/openapi-transformer-pipeline that referenced this issue Oct 2, 2023
This reverts commit 6f3f074.

The tagging of codecov/codecov-action@v4 was premature:
codecov/codecov-action#1089
Wait until v4 is out of beta to upgrade.

Signed-off-by: Kevin Locke <kevin@kevinlocke.name>
kevinoid added a commit to kevinoid/promise-nodeify that referenced this issue Oct 2, 2023
This reverts commit e9761c1.

The tagging of codecov/codecov-action@v4 was premature:
codecov/codecov-action#1089
Wait until v4 is out of beta to upgrade.

Signed-off-by: Kevin Locke <kevin@kevinlocke.name>
kevinoid added a commit to kevinoid/swagger-spec-validator that referenced this issue Oct 2, 2023
This reverts commit b7a9204.

The tagging of codecov/codecov-action@v4 was premature:
codecov/codecov-action#1089
Wait until v4 is out of beta to upgrade.

Signed-off-by: Kevin Locke <kevin@kevinlocke.name>
kevinoid added a commit to kevinoid/travis-status that referenced this issue Oct 2, 2023
This reverts commit 24ce306.

The tagging of codecov/codecov-action@v4 was premature:
codecov/codecov-action#1089
Wait until v4 is out of beta to upgrade.

Signed-off-by: Kevin Locke <kevin@kevinlocke.name>
AndrewChubatiuk pushed a commit to AndrewChubatiuk/VictoriaMetrics that referenced this issue Oct 3, 2023
@codingjoe
Copy link

FYI, some might have already merged the v4 upgrade and will see errors now. Removing releases and tags from GitHub is probably not a good practice. I would suggest returning the v4 tag, but with v3 code and jumping a version number. I know it's annoying for you, but the damage is done, and that likely the best path of mitigation for paying customers.

@schankam
Copy link

schankam commented Dec 28, 2023

@codingjoe I agree, also the setup doc is not up-to-date on the website and suggest using v4.

Screenshot 2023-12-28 at 16 48 44

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants