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

Wrong hash for cucumber releases? #223

Closed
ciaranmcnulty opened this issue Feb 17, 2021 · 6 comments · Fixed by #225
Closed

Wrong hash for cucumber releases? #223

ciaranmcnulty opened this issue Feb 17, 2021 · 6 comments · Fixed by #225

Comments

@ciaranmcnulty
Copy link
Contributor

We get the list by running

git ls-remote --tags https://github.com/cucumber/cucumber.git 

Which looks like:

580f185fb19285e42373c4cb2a88a8de66b308a3        refs/tags/cucumber-gherkin/v17.0.1
99706895df020c8d005bc6d32afe2fec691b8493        refs/tags/cucumber-gherkin/v17.0.2

And use that hash for the cucumber update.

However, when I look into the Github UI, that hash seems to be of the previous commit?

Screenshot 2021-02-17 at 10 34 48

So we are potentially getting the wrong hash. The cucumber release process means this is only missing a changeling and some version numbers, but could be quite dangerous in future if unresolved

@stof
Copy link
Member

stof commented Feb 17, 2021

this is due to how tags are done in the cucumber repo:
image

they have 2 different tags on different commits

@stof
Copy link
Member

stof commented Feb 17, 2021

I would say that this is probably a glitch on the cucumber side. I don't see why this would be intended.

@ciaranmcnulty
Copy link
Contributor Author

Aha so it should be a simple fix, thank you

@stof
Copy link
Member

stof commented Feb 17, 2021

well, the question is whether we even need to change something. the cucumber-gherkin/* tags are the one they use for the github releases.

@ciaranmcnulty
Copy link
Contributor Author

They don't contain the changelogs and I wanted to get that into the PR

@ciaranmcnulty
Copy link
Contributor Author

From what I can see, they release cucumber-gherkin which then prompts cukebot to update and sync a load of stuff, which tags gherkin and manages the subrepo stuff?

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

Successfully merging a pull request may close this issue.

2 participants