-
Notifications
You must be signed in to change notification settings - Fork 585
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
Added shorthash to git functions #2752
Conversation
@xperiandri what do you think :) ? |
@xperiandri Ah I see, I need to run fantomas on it to merge. I'll handle that after work today. |
@voronoipotato could you format your changes? Build fails |
@DanielRowe1 let's merge this and I will release |
Thanks! We use the shorthash at my work sometimes so this is pretty nice. |
I'll format it after work today. Thanks! |
fwiw I think Fantomas might just want empty lines adding before and after the new function |
I tried tweaking the formatting in a separate branch over at Numpsy@ad0da7c and the formatting step of the CI build worked - https://github.com/Numpsy/FAKE/actions/runs/7504044055 - though that intermitent macOS build failure happened in the integration tests. Not sure if I'd be able to update this PR though, or if it'd need a new one off my fork. |
@voronoipotato This addition has hopefully been merged in #2770 now |
@voronoipotato you can delete the branch |
Description
Grabs the shorthash