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

Add pager oncall and pager oncall <schedule> commands #5

Merged
merged 3 commits into from
May 19, 2015

Conversation

davidcornu
Copy link
Contributor

Removes the who's on call command in favour of pager oncall and pager oncall <schedule>.

Usage:

lita > lita help pager oncall
lita: pager oncall - List available schedules
lita: pager oncall <schedule> - Show who is on call for the given schedule
lita > lita pager oncall
Available schedules: Primary Ops, Secondary Ops, Primary Data, Secondary Data
lita > lita pager oncall Primary Ops
Example User (user@example.com) is currently on call for Primary Ops

@davidcornu
Copy link
Contributor Author

Fixes #4.

@coveralls
Copy link

Coverage Status

Coverage decreased (-5.44%) to 93.58% when pulling 77f74f5 on davidcornu:oncall into add887b on PagerDuty:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-6.64%) to 92.38% when pulling 0ab2a05 on davidcornu:oncall into add887b on PagerDuty:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-6.64%) to 92.38% when pulling 0ab2a05 on davidcornu:oncall into add887b on PagerDuty:master.

@esigler
Copy link
Contributor

esigler commented Feb 27, 2015

Fantastic! Thanks for sending this PR, will look over shortly.

teohm pushed a commit to teohm/lita-pagerduty that referenced this pull request Apr 20, 2015
@johntdyer
Copy link
Contributor

Any update here?

@esigler
Copy link
Contributor

esigler commented May 19, 2015

LGTM!

esigler added a commit that referenced this pull request May 19, 2015
Add pager oncall and pager oncall <schedule> commands
@esigler esigler merged commit c25c427 into PagerDuty:master May 19, 2015
@davidcornu
Copy link
Contributor Author

🎉

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.

4 participants