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

feat: full hash in commit.hbs for links #880

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

mrt181
Copy link

@mrt181 mrt181 commented Dec 6, 2021

This changes the link to a commit to use the full hash instead of the shortened hash that is used in the link preview.
see also: #102 (comment)

This changes the link to a commit to use the full hash instead of the shortened hash that is used in the link preview.
see also: conventional-changelog#102 (comment)
@bcoe
Copy link
Member

bcoe commented Dec 25, 2021

@mrt181 is the reason for this that the full short hash has does not work on some platforms?

@mrt181
Copy link
Author

mrt181 commented Dec 25, 2021

Yes, the short hash does not work with bitbucket and azure DevOps for example

@bcoe
Copy link
Member

bcoe commented Dec 30, 2021

@mrt181 this seems like a good fix 👌 but mind updating tests?

@mrt181
Copy link
Author

mrt181 commented Jan 10, 2022

@mrt181 this seems like a good fix 👌 but mind updating tests?

Hey, how can I run/ debug them in vs code? mocha sidebar doesn't seem to work?
nevermind got it to work by installing the missing module to the root

@mrt181
Copy link
Author

mrt181 commented Jan 10, 2022

@bcoe Those tests fail even when i revert my changes?

@armano2
Copy link
Contributor

armano2 commented Apr 1, 2024

Yes, the short hash does not work with bitbucket and azure DevOps for example

depends on has shorten version is sometimes broken on github (tho i noticed it only 2 times on github enterprise)

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

Successfully merging this pull request may close these issues.

None yet

3 participants