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
add the no-strip
flag to the clip command
#9216
Conversation
ansi strip removes tab symbols, which is sometimes useful (for example, when using the `... | to tsv | clip` command). Therefore, I added a flag to prevent using `ansi strip` in the clip command.
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #9216 +/- ##
==========================================
- Coverage 68.92% 68.54% -0.38%
==========================================
Files 635 636 +1
Lines 102041 102281 +240
==========================================
- Hits 70329 70106 -223
- Misses 31712 32175 +463 |
so this is ready for review, isn't it @maxim-uvarov ? 😋 |
@amtoine, yep, it is ready for review! 🙏 |
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.
yep, it is ready for review!
then it looks just fine to me, the CI passes and it works as before locally 👌
I'm just learning how Git works here.
you mean GitHub? 😋
Thank you for your help!
anytime 😌
Description
ansi strip
(used in the clip command by default) removes tab symbols, which is sometimes not useful (for example, when using the[[a b]; [1 2]] | to tsv | clip
command). Therefore, I added a flag to prevent usingansi strip
in the clip command.User-Facing Changes
Tests + Formatting
After Submitting