Skip to content

Add a confirmation prompt for sensitive commands #388

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

Merged
merged 1 commit into from
Apr 11, 2022

Conversation

duboiss
Copy link
Contributor

@duboiss duboiss commented Apr 5, 2022

No description provided.

@duboiss duboiss force-pushed the feature/confirmation branch from 2d5a2c1 to 7185cea Compare April 5, 2022 15:47
@duboiss duboiss force-pushed the feature/confirmation branch from 7185cea to 7bd5397 Compare April 8, 2022 10:20
@codecov
Copy link

codecov bot commented Apr 8, 2022

Codecov Report

Merging #388 (7bd5397) into master (d37aecb) will decrease coverage by 0.18%.
The diff coverage is 94.11%.

@@             Coverage Diff              @@
##             master     #388      +/-   ##
============================================
- Coverage     97.11%   96.93%   -0.19%     
- Complexity      338      339       +1     
============================================
  Files            41       41              
  Lines           972      979       +7     
============================================
+ Hits            944      949       +5     
- Misses           28       30       +2     
Impacted Files Coverage Δ
src/Command/UninstallCommand.php 100.00% <ø> (ø)
src/Command/UpdateCommand.php 100.00% <ø> (ø)
src/Command/Database/RestoreCommand.php 84.61% <75.00%> (-3.96%) ⬇️
src/Command/Database/ResetCommand.php 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d37aecb...7bd5397. Read the comment docs.

@ajardin ajardin merged commit 5246c43 into origamiphp:master Apr 11, 2022
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