[GitHub] [accumulo] ddanielr opened a new pull request, #3704: Standardizes DeleterFormatter confirm prompt

["ddanielr (via GitHub)" <gi...@apache.org>]

ddanielr opened a new pull request, #3704:
URL: https://github.com/apache/accumulo/pull/3704

   Converts DeleterFormatter's confirmation prompt to use Shell.confirm


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [accumulo] ctubbsii merged pull request #3704: Standardizes DeleterFormatter confirm prompt

["ctubbsii (via GitHub)" <gi...@apache.org>]

ctubbsii merged PR #3704:
URL: https://github.com/apache/accumulo/pull/3704


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [accumulo] ddanielr commented on pull request #3704: Standardizes DeleterFormatter confirm prompt

["ddanielr (via GitHub)" <gi...@apache.org>]

ddanielr commented on PR #3704:
URL: https://github.com/apache/accumulo/pull/3704#issuecomment-1682982101

   > I added my code review suggestions as a commit to this PR. I think it's good to go.
   
   I like the cleaner logic changes and general cleanup of the Test class.
   I'll go ahead and merge this PR.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org