timetrace delete commands: Default to no when asking for confirmation

dominikbraun / timetrace

A simple CLI for tracking your working time.

Apache License 2.0

679 stars 75 forks source link

Closed jwnpoh closed 3 years ago

jwnpoh commented 3 years ago

This PR addresses #110.

askForConfirmation() now defaults to no unless the input is specifically "y" or "Y".
added askForConfirmation() to deleteProjectCommand

dominikbraun commented 3 years ago

Thanks for solving this, @jwnpoh! Your change will be included in the next release.

jwnpoh commented 3 years ago

Thrilled to have my first merged PR! Thanks for the opportunity!

dominikbraun commented 3 years ago

This PR is included in timetrace v0.11.1.