-
Notifications
You must be signed in to change notification settings - Fork 124
Removed macros. #1457
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
Removed macros. #1457
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1457 +/- ##
==========================================
- Coverage 98.57% 98.50% -0.07%
==========================================
Files 20 20
Lines 5749 5566 -183
==========================================
- Hits 5667 5483 -184
- Misses 82 83 +1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
|
||
return statement | ||
|
||
def _input_line_to_statement(self, line: str, *, orig_rl_history_length: Optional[int] = None) -> Statement: |
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.
I forgot how much avoidable complexity macros added. Nice work surgically removing them. Everything here LGTM.
I see this is closed, so I guess there's no hope for macros? I'm not sure where to put this comment, I posted the same thing in a discussion on macros and tab completion. I'll paste it here too, just to be sure you see it: The great thing about macros for me is it lets my users create their own bespoke commands and toolkit. Not sure if it's too late to chime in on macros. I find them very useful despite lack of autocomplete. Some command can call sub-commands and it can get verbose, e.g. the find command can run a command string given by --cmd CMD on each matched object. I will work on improving syntax of the find command, but right now I'd be very sad to lose macros, they cut typing effort significantly (I was planning to include macros in upcoming demo videos. Now I'm not sure what to do.) Here is a list of my favourite macros. And I'm discovering new tricks all the time gn
|
No description provided.