Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Matrix reporter #421

Open
wants to merge 3 commits into
base: master
from

Conversation

@reivilibre
Copy link

reivilibre commented Oct 8, 2019

This adds a reporter compatible with the Matrix.org protocol, as used in e.g. Riot.im.

reivilibre added 3 commits Jan 8, 2019
Signed-off-by: Olivier Wilkinson (reivilibre) <olivier@librepush.net>

try:
# markdown2 is an optional dependency which provides better formatting for Matrix.
from markdown2 import Markdown

This comment has been minimized.

Copy link
@thp

thp Dec 3, 2019

Owner

This optional dependency should be mentioned in the README.md file.

@@ -614,3 +624,110 @@ def submit_to_slack(self, webhook_url, text):

def chunkstring(self, string, length):
return (string[0 + i:length + i] for i in range(0, len(string), length))


class MarkdownReporter(ReporterBase):

This comment has been minimized.

Copy link
@thp

thp Dec 3, 2019

Owner

Can the MarkdownReporter share some/more code with the TextReporter? Seems like there's some code duplication there now.

Copy link
Owner

thp left a comment

Good PR, but needs some cleaning up -- see comments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.