Phoenix Himself

Results 280 comments of Phoenix Himself

Yeah, At first I wanted to make Amber feel more JavaScriptish so I decided to use the `bc` everywhere. However this creates a limitation and dependency on the `bc` command....

@eXamadeus Thank you for your point of view. I also think that we need to pivot the vision a little and simply add the Integer type to Amber. The closer...

> #95 mentions checking for `sed`, `[[` and `bc`, but `[[` IIRC is specifically a Bash-ism. Looking at some Docker images like `amazonlinux` and `ubuntu`, they include `sed` but not...

@b1ek That's a fantastic idea, although why not make the regular comment `//` stay in the bash code? I think that we could add a flag `--release` that will eradicate...

> Also thats somewhat of a breaking change as compared to creating a new syntax rule. I think we're early enough to introduce such changes. It's better now than later.

Really good job implementing this @b1ek! Here is some bug that I found. When I ran this code that used a standard library function `split` it failed with this error:...

@boushley @arapower can you take a look at this?

> Yeah, looks pretty simple > > > > ![image](https://github.com/Ph0enixKM/Amber/assets/57577711/ab0ca618-d6d2-4036-a1b4-d682de299c6f) > > This doesn't seem to match the following string though: ``` IFS="hello world" read --arg1 --arg2 ```

We should continue this discussion on Github Discussions that I'll create this evening. This PR is great and I think we can leave the basic RDC for Amber itself. I'd...