-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
Remove all leading $ not just one #1039
Conversation
This would also remove any leading spaces, like |
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.
Cool!
Can you please include some tests for that?
Thanks for contributing!
7beecad
to
d0a4970
Compare
Added some tests, let me know what you think. and also updated .gitignore to exclude venv, and requirements.txt which was missing some packages |
@scorphus ping |
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.
Thanks, @theslimshaney! Please mind my comments below.
Merged in 9201ce7. Thanks, @theslimshaney. |
Related to #996
This way we can give less fucks if we have more $
Before
After