BUG: Remove spare `f` in an f-string

SlicerDMRI / whitematteranalysis

White matter tractography clustering and more...

https://dmri.slicer.org/whitematteranalysis/

Other

69 stars 33 forks source link

Closed jennydaman closed 9 months ago

jennydaman commented 9 months ago

Remove a spare f when redirecting the output of a 3D Slicer command call using an f-string in the transform hardening script.

Inadvertently introduced in commit ac6040f.

jhlegarreta commented 9 months ago

@jennydaman Thanks for the patch set !

Can you please amend your commit and push force with the following changes?

Add the BUG: commit prefix to the commit subject line.
Add a commit message body with a more elaborate message, e.g. explaining the failure this patch set fixes.

You can find some examples of accurate messages in the commit history. This blog post is very enlightening: https://cbea.ms/git-commit/amp/

jennydaman commented 9 months ago

@jhlegarreta I force-pushed a better commit message.

jhlegarreta commented 9 months ago

@jhlegarreta I force-pushed a better commit message.

@jennydaman Only the commit message body was changed; there is no commit message body. I will amend and push force.

jhlegarreta commented 9 months ago

Thanks for having detected and fixed this @jennydaman.

Merging.