Daniele Scasciafratte

icon indicating a website link https://daniele.tech icon indicating an email [email protected]

icon company Codeat SRLS icon location Rieti, Italy icon location CTO @CodeAtCode, Mozilla Volunteer and ex-WordPress Core Contributor. Open Source Multiversal Guy with an addiction in contributing and evangelism.

Results 1200 comments of Daniele Scasciafratte

[Feature] echo to stderr?

The proposal for stdout/stderr/io I think is perfect and we can integrate it in the other functions that amber offers.

[Feature] echo to stderr?

I am for the second approach, in this way it is more functional programming and we can implement quickly in the stdlib. About the keyword approach it is the case...

[Bug] Shorthand assignment operator `+=` is not supported by Bash 2

Wondering if it is better a CLI parameter or a comment in the amber file that specify the bash version to support.

[Feature] Call Amber function from Bash script

It should make sense to not rename functions and should be fine. Reading the bash generated it is just that, I think that the renaming is used only for the...

[Feature] Call Amber function from Bash script

Maybe `interpreter` so in base it is compiled for `sh`, `bash` does automatically?

[BUG] using command modifiers with builtin function

I can confirm the issue, the discord bot instead return 1.

[BUG] using command modifiers with builtin function

the issue is still there

[BUG] using command modifiers with builtin function

it is still there

[Bug] __AS=$? on every command also if the status isn't checked

I think that should be enough to check on https://github.com/Ph0enixKM/Amber/blob/69e96ae3867091bcd374ff3cf42909324e73646e/src/modules/expression/literal/status.rs#L28 if the command is unsafe to not print that

[Bug] __AS=$? on every command also if the status isn't checked

If the status is not used after why add code anyway? This is kind of a compiler optimization to generate only useful code.