Daniele Scasciafratte

Results 1200 comments of Daniele Scasciafratte

I think that we need to define built-in should be one shot `actions`, instead ORM style should be more complex commands with various parameters. The ORM style requires a Class...

The ticket is not yet closed as the topic was to add more builtin commands and there was a PR linked that doesn't involve that.

RIght now we are missing: * `rm` * `exit` I think that we can add `touch` too. In this way we have the majority of builtin for our tests. Maybe...

For that one I think that we have to wait for @Ph0enixKM.

I think that you can go with a PR with the CLI parameter to not print it. Consider also that Amber is not stable so we can change the Bash...

You can do a PR we are working on various stuff right now that have more priority. I think that as we are in alpha is fine a parameter and...

I think that it is better to use `bc` for calculation. It is part of the gnu command set https://www.gnu.org/software/bc/manual/html_mono/bc.html and it is everywhere and we will have the bash...

That line also generate vvarious shellcheck issues: ``` if [ "$(echo "$(echo ""$__AF_date_compare3_v0__4_16"" '==' "$(echo "" '-' "1" | bc -l | sed '/\./ s/\.\{0,1\}0\{1,\}$//')" | bc -l | sed...

so checking the code it is there to remove trailing zero from the expression generated by `bc`.

I think https://askubuntu.com/questions/217570/bc-set-number-of-digits-after-decimal-point We can use `bc` to generate without trailing zero using the `scale` variable and we can remove the `sed` command. I think that the issue starts from...