Closed voronoipotato closed 4 months ago
@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
@voronoipotato
I tried tweaking the formatting in a separate branch over at https://github.com/Numpsy/FAKE/commit/ad0da7ca26d98fc5f07018e490aab0e404419a8c 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