Conversation
de05ccc
to
146dab7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just two small comments 🏑
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nothing to add to @NunoAlexandre observations, happy to approve once addressed. Test needs to be looked at.
I'm wondering whether we should also add this to the handle input in onboarding as well to keep things consistent. That input also disallows spaces. \cc @brandonhaslegs @juliendonck |
yes :) |
Other than the skipped tests works and looks great! |
Co-authored-by: Rūdolfs Ošiņš <rudolfs@osins.org>
Replaces spaces with dashes for new project names; unit-tests name formatting
Closes #529