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

Still retrieve git-version if using gitdir [no more ¯\_(ツ)_/¯] #250

Merged
merged 1 commit into from
Sep 9, 2021

Conversation

XANOZOID
Copy link
Contributor

@XANOZOID XANOZOID commented Sep 8, 2021

Support an alternative git structure that uses gitdir: instead of a directory with git information.

This fixes an issue, for me at least, where Khamake was only printing out ¯\_(ツ)_/¯ for the version instead of the actual git version.
I'm not sure if anything about git changed recently, but my project doesn't utilize git folders it just contains information in the .git file of where the relative git directory is.

@XANOZOID XANOZOID changed the title Still retrieve git-version if using gitdir Still retrieve git-version if using gitdir [no more ¯\_(ツ)_/¯] Sep 8, 2021
@RobDangerous
Copy link
Member

There's a small style-issue to fix, would you do the honors?

@XANOZOID
Copy link
Contributor Author

XANOZOID commented Sep 9, 2021

There's a small style-issue to fix, would you do the honors?

Ahh, okay - taken care of! How can I check that on my side before submitting a pr, next time, if I may ask?

@RobDangerous
Copy link
Member

Next time the GitHub-check will run automatically. To check it locally I use a tsline extension for vscode but you can also just run tsline manually.

@RobDangerous RobDangerous merged commit c18fa71 into Kode:master Sep 9, 2021
@RobDangerous
Copy link
Member

*tslint

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 this pull request may close these issues.

2 participants