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

Latest release is missing the v #100

Closed
bmaupin opened this issue Oct 7, 2021 · 8 comments
Closed

Latest release is missing the v #100

bmaupin opened this issue Oct 7, 2021 · 8 comments

Comments

@bmaupin
Copy link

bmaupin commented Oct 7, 2021

This is pretty trivial so feel free to close it, but the latest release is 1.1.3 whereas all the other release tags start with a v, e.g. v1.1.2.

Thanks!

@hartwork
Copy link

hartwork commented Dec 25, 2021

My vote for adding a copy named v1.1.3 with v too, e.g. because I am worried that mixing schemes will confuse GitHub Dependabot thank you!

CC @afinetooth

@jrfnl
Copy link

jrfnl commented Dec 26, 2021

Could this be why the below does not work ?

uses: coverallsapp/github-action@v1

It is generally strongly recommended to always "lock" actions to a version to prevent CI builds to unexpectedly start failing due to a new major of an action having been released. That is currently not possible with this action.

@jrfnl
Copy link

jrfnl commented Dec 26, 2021

Related to / duplicate of #94

@hartwork
Copy link

hartwork commented Dec 26, 2021

@jrfnl @v1 is a moving Git tag that other repos keep moving whenever they create a new v1.x.y tag, by themselves. Ideally, Git tags never move, so it's arguably a hack. The reason it doesn't work here is because there is no tag named v1 at https://github.com/coverallsapp/github-action/tags .

@jrfnl
Copy link

jrfnl commented Dec 26, 2021

@hartwork Thanks for the explanation. Sounds like something which should be solved.

@prescottprue
Copy link

prescottprue commented Jan 20, 2022

@jrfnl I'm not seeing any CI in the repo currently if you are implementing in a Github Actions workflow, I've done that with a simple step to push the change to a branch after releasing (this example grabs version from the package.json since you seem to be managing that manually):

      - name: Push updates to branch for major version
        # If a new version is published, i.e. v1.2.3 then this step will update
        # branch "v1" to this commit.
        env:
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
        run: |
          git push https://x-access-token:${GITHUB_TOKEN}@github.com/${GITHUB_REPOSITORY}.git HEAD:refs/heads/v$(cat package.json | jq '.version')

Using semantic-release like Cypress does with their Github Action makes it even cleaner since you can detect if a version was released:

      - name: Semantic Release
        uses: cycjimmy/semantic-release-action@v2
        id: semantic
        env:
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

      - name: Push updates to branch for major version
        # If a new version is published, i.e. v1.2.3 then this step will update
        # branch "v1" to this commit.
        if: steps.semantic.outputs.new_release_published == 'true'
        env:
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
        run: |
          git push https://x-access-token:${GITHUB_TOKEN}@github.com/${GITHUB_REPOSITORY}.git HEAD:refs/heads/v${{steps.semantic.outputs.new_release_major_version}}

I'm open to help setting up the CI tooling for release as well as commit linting if you are interested, but up to you since semantic-release is more strict than some projects want

@jrfnl
Copy link

jrfnl commented Jan 20, 2022

@prescottprue I think your suggestions are great, however, I'm not a maintainer of this repo, just a user 🤷🏻‍♀️

I imagine @nickmerwin may welcome your help though ?

@mrexox
Copy link
Contributor

mrexox commented Feb 27, 2023

Hey! There is a new release with the fix, and also there is a v1 release which is finally what many people wanted.

@mrexox mrexox closed this as completed Feb 27, 2023
jrfnl added a commit to PHPCSStandards/PHPCSUtils that referenced this issue Mar 4, 2023
The `coverallsapp/github-action` action runner has (finally) created a named branch for the 1.x series, so let's use that.

Ref: coverallsapp/github-action#100
jrfnl added a commit to PHPCompatibility/PHPCompatibility that referenced this issue Mar 8, 2023
The `coverallsapp/github-action` action runner has (finally) created a named branch for the 1.x series, so let's use that.

Ref: coverallsapp/github-action#100
jrfnl added a commit to PHPCompatibility/PHPCompatibility that referenced this issue Mar 8, 2023
The `coverallsapp/github-action` action runner has (finally) created a named branch for the 1.x series, so let's use that.

Ref: coverallsapp/github-action#100
jrfnl added a commit to PHPCompatibility/PHPCompatibility that referenced this issue Mar 8, 2023
The `coverallsapp/github-action` action runner has (finally) created a named branch for the 1.x series, so let's use that.

Ref: coverallsapp/github-action#100
jrfnl added a commit to sirbrillig/phpcs-variable-analysis that referenced this issue Mar 12, 2023
The `coverallsapp/github-action` action runner has (finally) created a named branch for the 1.x series, so let's use that.

Ref: coverallsapp/github-action#100
sirbrillig pushed a commit to sirbrillig/phpcs-variable-analysis that referenced this issue Mar 12, 2023
* GH Actions: used a named branch for coverallsapp

The `coverallsapp/github-action` action runner has (finally) created a named branch for the 1.x series, so let's use that.

Ref: coverallsapp/github-action#100

* GH Actions: fix CI

Grrr....

When PHPUnit has been installed on a high PHP version, some of the dependencies of PHPUnit will now be installed in versions not compatible with PHP 7.4, which blocks the install of the Coveralls package.

Installing PHP Coveralls globally instead should fix it.

I just wish PHP Coveralls would finally release a version compatible with PHP >  8.0....

---------

Co-authored-by: jrfnl <jrfnl@users.noreply.github.com>
jrfnl added a commit to PHPCSStandards/PHPCSExtra that referenced this issue Mar 20, 2023
The `coverallsapp/github-action` action runner has (finally) created a named branch for the 1.x (and the 2.x) series, so let's use that.

Ref: coverallsapp/github-action#100
jrfnl added a commit to php-parallel-lint/PHP-Console-Highlighter that referenced this issue Jul 16, 2023
The `coverallsapp/github-action` action runner has (finally) created a named branch for the 1.x series, so let's use that.

Ref: coverallsapp/github-action#100
jrfnl added a commit to php-parallel-lint/PHP-Console-Highlighter that referenced this issue Nov 23, 2023
The `coverallsapp/github-action` action runner has (finally) created a named branch for the 1.x series, so let's use that.

Ref: coverallsapp/github-action#100
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

5 participants