kha7iq / pingme

PingMe is a CLI which provides the ability to send messages or alerts to multiple messaging platforms & email.
https://pingme.lmno.pk
MIT License
771 stars 43 forks source link

add support for matrix server #65

Closed ahdekkers closed 2 years ago

ahdekkers commented 2 years ago

Description

Added matrix service

Fixes #61

Type of change

Please delete options that are not relevant.

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

Checklist:

ahdekkers commented 2 years ago

Let me know if I should make any changes, @kha7iq

kha7iq commented 2 years ago

@ahdekkers Sorry for the delay , i am swamped with some work these days. Just a small thing, can you please confirm if you have tested sending messages to matrix server with the changes ? Also can you please lint the code make lint and see if everything is good. I will merge afterwards

ahdekkers commented 2 years ago

Hey, no worries. Yeah sending matrix messages work, I only used official matrix servers for the testing. The make lint isn't working for me - for some reason it tells me: golangci-lint run ./... ERRO Running error: context loading failed: no go files to analyze make: *** [lint] Error 5 Same thing when I run golangci-lint run service/matrix. Have you encountered this before and does it work for you?