Renato Alves
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