Results 157 comments of Renato Alves

I don't think I can get rid of the 2 codeclimate issues without making the code harder to read. @attzonko do you think this is acceptable? Also I ended up...

I'll try to merge this and #326 and address test failures after as there's a little too much in flux.

Hi @saf6260. Thanks a lot for the pointers! This is gold and a massive time saver!!! Would be great to have a chat. We don't want to distract you from...

Hum... the link seems to work for me. Could you maybe try a different browser (without addons) or visiting the link in incognito mode? Perhaps some screenshots will help us...

I was able to work on this for another project and works (locally) with the `mattermost/mattermost-preview` container. Here's a possibly useful extract: ```bash TARGET="container_name" MM="docker exec $TARGET mattermost --config=mattermost/config/config_docker.json" MMCTL="docker...

Integration tests already do this so closing.

Some partial code that could be used to implement this. This permits extracting `args` and `kwargs` from a regexp. ```python named_args = set(matcher.groupindex.values()) groups = list([group for i, group in...

Some significant changes from #212 - Tests are not yet in sync. More on that in the next push: * `Plugin.help` and `Plugin.get_help_string` is now `HelpPlugin.help` and `HelpPlugin.get_help_string`. * The...

I think this is ready for a final of review.

This PR will also close #165