-
Notifications
You must be signed in to change notification settings - Fork 643
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(cli): add ability to escape at-prefixed parameter #2513
Conversation
Codecov Report
📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more @@ Coverage Diff @@
## main #2513 +/- ##
==========================================
- Coverage 96.15% 96.13% -0.02%
==========================================
Files 87 87
Lines 5663 5665 +2
==========================================
+ Hits 5445 5446 +1
- Misses 218 219 +1
Flags with carried forward coverage won't be shown. Click here to find out more.
|
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 a lot @pyhedgehog for this and for adding the tests. LGTM just a small suggestion for the docs.
LGTM. @nejch I'll let you decide when to merge. |
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 @pyhedgehog and @JohnVillalovos!
Fixes #2511