arturo-lang / arturo

Simple, expressive & portable programming language for efficient scripting

http://arturo-lang.io

MIT License

675 stars 29 forks source link

Fix bug with System\args + add cmdline-related unit-tests #1558

Closed drkameleon closed 5 months ago

drkameleon commented 5 months ago

Description

Fixes #1448

Type of change

[x] Code cleanup
[x] Unit tests (added or updated unit-tests)
[x] Bug fix (non-breaking change which fixes an issue)

drkameleon commented 5 months ago

@RickBarretto I so love.. 🪟🪟. Apparently, args wasn't suddenly "broken". It's that it's just not working on my no.1 favorite OS... (what a surprise, I cannot believe my eyes! lol)

P.S. If you don't mind, I'll ask for a final review if you can - before merging - but first let's see how I can get it to work! 🤦

drkameleon commented 5 months ago

@RickBarretto The PR seems quite solid. If/when you have time, I would be grateful for a review! 😉

RickBarretto commented 5 months ago

I'm having a look right now. Give some minutes, and I give you the feedback.

drkameleon commented 5 months ago

Working perfectly! 🙂 Well done work! 🚀

Thanks!! :)

drkameleon commented 5 months ago

Ready to merge! 🚀