Thanabodee Charoenpiriyakij

Results 190 comments of Thanabodee Charoenpiriyakij

I found that the formatter will return this error when ElixirLS are still compiling the source. If your source contains small dependencies. I guess that we cannot get formatter opt...

Hi, all. I've start resolving test error step by step. I start open PR to resolve the compiling error here https://github.com/benwilson512/elixir-ls/pull/1 (it's PR against this PR).

I has been sending a pr to start tackle the test suite here https://github.com/benwilson512/elixir-ls/pull/1. I’m not sure someone take a look? btw, it seem pr got merge conflict. I think...

I’m would like to take over it if you’re ok. :)

@arcticicestudio Thanks for your reply. I'll take a look what I can do with this. :)

I just update on my research. It seems we need to decorate `smerge` theme. In the image below, I custom `smerge-refined-added` to use `nord7`. It's works but the color look...

Add 3 lines for `smerge` to the same `diff` looks better. ``` ;; > smerge `(smerge-refined-added ((,class (:foreground ,nord14)))) `(smerge-refined-changed ((,class (:foreground ,nord13)))) `(smerge-refined-removed ((,class (:foreground ,nord11)))) ``` ![Screen Shot...

Compare to diff in `magit-show-commit`. The result from previous comment looks very weired. 🤔 ![Screen Shot 2563-10-03 at 21 24 41](https://user-images.githubusercontent.com/484530/94993983-dfb46a00-05be-11eb-9371-06ffa8b0d10a.png)

Or if we use `doc` sub-command instead? ``` earthly doc +target ```

@williammartin @BooleanCat first comment got fixed. but the second comment I'm stuck with gomega . I already add the detail in second comment. Please take a look. Thanks