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

adding support to victorops #401

Merged
merged 29 commits into from
Aug 23, 2021
Merged

adding support to victorops #401

merged 29 commits into from
Aug 23, 2021

Conversation

alonm-totango
Copy link
Contributor

No description provided.

F541 f-string is missing placeholders
Copy link
Owner

@liiight liiight left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this! I hope to get the base branch test stable in the next few days, I'll let you know when to pull so that tests can pass.

Also, please add tests if possible

notifiers/providers/victorops.py Outdated Show resolved Hide resolved
notifiers/providers/victorops.py Outdated Show resolved Hide resolved
notifiers/providers/victorops.py Outdated Show resolved Hide resolved
notifiers/providers/victorops.py Outdated Show resolved Hide resolved
notifiers/providers/victorops.py Outdated Show resolved Hide resolved
notifiers/providers/victorops.py Outdated Show resolved Hide resolved
source/providers/victorops.rst Outdated Show resolved Hide resolved
@matancha
Copy link

matancha commented Aug 4, 2021

This would be very useful! Looking forward to it

Copy link
Owner

@liiight liiight left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a few more tweaks :-)

notifiers/providers/victorops.py Outdated Show resolved Hide resolved
notifiers/providers/victorops.py Outdated Show resolved Hide resolved
notifiers/providers/victorops.py Outdated Show resolved Hide resolved
tests/providers/test_victorops.py Show resolved Hide resolved
alonm-totango and others added 5 commits August 5, 2021 13:38
and using lower case for message_type field
and using lower case for message_type field
and using lower case for message_type field
notifiers/providers/victorops.py Outdated Show resolved Hide resolved
source/providers/victorops.rst Outdated Show resolved Hide resolved
alonm-totango and others added 2 commits August 8, 2021 19:15
raising errors for unsupported value of annotations
raising errors for unsupported value of annotations
# Conflicts:
#	notifiers/providers/victorops.py
Copy link
Owner

@liiight liiight left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a minor comment about that description, but looks good to go to me!

notifiers/providers/victorops.py Show resolved Hide resolved
notifiers/providers/victorops.py Show resolved Hide resolved
@liiight
Copy link
Owner

liiight commented Aug 23, 2021

I'll rebase and merge this after #404 is done, i'm doing a bunch of maintenance there

@liiight liiight merged commit 68a0f9f into liiight:master Aug 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants