Skip to content
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

fix: making false the default value of the delete command prompt #411

Merged
merged 2 commits into from
Jul 27, 2023

Conversation

tuanphamcybozu
Copy link
Contributor

@tuanphamcybozu tuanphamcybozu commented Jul 26, 2023

Why

  • Making false the default value of the delete command prompt

What

  • Change default value from true to false

How to test

  • Run delete command and check the default answer

Checklist

  • Read CONTRIBUTING.md
  • Updated documentation if it is required.
  • Added tests if it is required.
  • Passed pnpm lint and pnpm test on the root directory.

@tuanphamcybozu tuanphamcybozu marked this pull request as ready for review July 26, 2023 09:58
@tuanphamcybozu tuanphamcybozu requested review from a team and tasshi-me and removed request for a team July 26, 2023 09:58
@tuanphamcybozu tuanphamcybozu changed the title fix: making false is the default value of prompt of delete command fix: making false the default value of the delete command prompt Jul 27, 2023
Copy link
Contributor

@hung-cybo hung-cybo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@hung-cybo hung-cybo merged commit 2641ec3 into main Jul 27, 2023
@hung-cybo hung-cybo deleted the fix/change-default-prompt-value branch July 27, 2023 09:03
This was referenced Jul 27, 2023
This was referenced Jan 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants