Adrian Ramirez Fornell

Results 14 comments of Adrian Ramirez Fornell

The idea plugin should provide the compiler plugin to the Intellij analyzer IIRC. Apart from that it's correct

As it is on this PR, `merge` serves the same purpose than `add` because if, continuing with your example, `a.merge(b)` doesn't do anything because `b` has to be a `JsObject`....

I agree to move them to `JsObject` but I would like the opinion of @nomisRev here too

@nomisRev @raulraja thoughts?

Related to https://github.com/47deg/helios/pull/69#discussion_r288890786 We can also move all the Js numeric models to internal models because we already have a public API on `JsNumber` which maps each numeric type to...

The last part is already done in travis but yes I agreed. I suppose it's time for Hood-Actions

Could you try to build without clean? It seems like this is part of a known issue where we need to look into the dependency graph to generate the code...

I suggest to change it to: ``` // A list that starts with `2` case 2 :: xs => … ```

Here you can see a little example on scala: ![scala](https://user-images.githubusercontent.com/15971742/114420215-6b353600-9bb4-11eb-9000-42746e5bd537.png) and the same in python: ![python](https://user-images.githubusercontent.com/15971742/114420225-6cfef980-9bb4-11eb-874a-4d6600039999.png) and this is how I see the same scala code on IntelliJ: ![intellij scala](https://user-images.githubusercontent.com/15971742/114420885-075f3d00-9bb5-11eb-8565-bad4580bf749.png)...

Adding a color for the a `.` following a name and optionally `(` or `{` looks awesome. I don't care to wait for scala 3 to make mandatory the parenthesis...