Alberto Sartori

Any news on this? I have the same problem being not able to ping or remote_console to my app with sname because of the inverted if statement in this PR

Yes, I'm not using it directly. I just tracked down the problem I have to this piece of code. I'm using graphite `smartSummarize` function to get daily stats from an...

I just saw this PR, but yesterday I opened an analogue PR for the `jwt` plugin (#2988). Since it is exactly the same change with exactly the same reasons I...

Maybe yes but in that case the plugin that needs to access the token from `ngx.ctx` can implement its own logic for retrieving one of the possible many tokens. If...

According to [this](https://code.visualstudio.com/api/extension-guides/command#command-uris), I think the problem is that commands and sometimes their arguments should be URI encoded. I tried as a proof of concept to replace the [this code](https://github.com/microsoft/codetour/blob/441e9a558cbbbf0e24dca5900ee7b16f80b3ab45/src/player/index.ts#L62)...

Yes, it is a good idea

Also because I made it work by inserting the absolute path of the file to open, we should also add some logic to prepend the workspace root directory if the...

One option is to add the `include_apps` option of `covertool` also to the mix coverage tool. Now it is available only for rebar.

Ideally we should cover everything :-) If we add a new Erlang function and the corresponding Elixir wrapper we should cover it, there can be typos like swapping arguments o...

I'm used to work with 100% coverage and PR blocked if the coverage decreases. This implies covering also code path that seems more "trivial" but it happened more than once...