before option not working anymore about fastest HOT 10 CLOSED

@matlar83 can you try and see if https://github.com/liuggio/fastest/tree/input_args_parsing fix this?
I've only fixed args parsing, the "older" issues with --before should not be trackled here.
Let me know please.
Thank you.

from fastest.

@DonCallisto, it does not fix it, according to my tests.
And it seems not to run the "execute" command anymore.

Performed test:

> rm -f before.txt test.txt
> ls | fastest --before="echo ciao >> before.txt" "echo test {} >> test.txt"
> ls -l before.txt test.txt
ls: cannot access 'before.txt': No such file or directory
ls: cannot access 'test.txt': No such file or directory

Can you reproduce it?
Thank you

from fastest.

I confirm, -b/--before isn't working

from fastest.

I was able to reproduce.
With my fix, I'm not using the StringInput the right way: as a matter of fact, it expects to match a certain input, so a first try is to copy/paste the function that parse input string directly in fastest but with no luck.
I'm not sure why the command is not interpreted in the right way.
Sadly last week, I had no time to investigate deeper.
I'll try this week but I really don't know if I'll have time.

If someone would like to help (with suggestions I gave before) is welcome to do so.

from fastest.

@matlar83 @Brightside56 sorry for the long delay, I was quite busy.
I've updated my fork https://github.com/liuggio/fastest/tree/input_args_parsing

Could you give it a shot and let me know if it's ok for your projects?
Mine tests were fine.

from fastest.

If no one will give a feedback, I'm gonna merge the PR within this week and release a new tagged version in order to let everyone use new code.
I'm pretty confident it's gonna be fine but a double check would be great.
Thanks in advance @matlar83, @Brightside56 and everyone will check this.

from fastest.

Sorry @DonCallisto, I don't have time to test it :-(

I hope I will have in the next weeks, but I'm not sure

from fastest.

Don't worry. I've used your test cases as guidelines and it works like a charm, so I'm pretty sure we won't have surprise here.
After all, this taken me 4 months, it won't be a problem if you can't try it.

from fastest.

I've released v1.7.1 https://github.com/liuggio/fastest/releases/tag/v1.7.1
It should fix the bug.

from fastest.

Gonna close it for the moment.
Feel free to comment if something's wrong and I'll reopen it.

from fastest.