whitespace not allowed in execute selection?

[pbrightly]

Is there any reason whitespace should not be allowed in execute_selection()?  I tend to keep all my notes using Slick, and I frequently keep some handy commands in my notes, allowing me to simply press Alt-= on any line to execute it.
I also like to format my text by levels of indentation using tabs and/or spaces.  Unfortunately, that causes execute_selection() to fail with a bad return code.  Easy fix, I just put strip() around the string and reloaded the module.  Should that be official? Or is there some reason not to do that?

PS. When I modify one of Slicks modules rather than my own, how is that effected on my next update?  Does it get overwritten?

[Clark]

What command are you executing? I tried adding leading space for a couple different commands and they both worked.

[pbrightly]

I've had many different things in my notes, often "e" or a "diff" command of ugly long paths to check my work against known-good files.  Make sure the leading space is tabs.  Two in the case below, if my paste is identical.

Code: [Select]

e .shellcheckrc
e tools/bin/common.sh
e tools/bin/logging.sh
e tools/bin/utils.sh
e tools/bin/common_example.sh
Reminder: I'm on Windows. I like using unix style paths as they work, and I can cut/paste to my Ubuntu command line.

[Clark]

Ah. The problem is leading tabs. Thanks

We are working on 25.0.2 at the moment. I'll add a hot fix for 25.0.2 since I think it's too late to add another fix to 25.0.2