Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

Adding teams support #23

Merged
merged 7 commits into from
May 5, 2019
Merged

Conversation

sedan07
Copy link
Contributor

@sedan07 sedan07 commented May 4, 2019

Pull Request Template

Description

Adds support for the teams API endpoint as documented here: https://grafana.com/docs/http_api/team/

Have you written tests?

  • Yes
  • No

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings

@codecov
Copy link

codecov bot commented May 4, 2019

Codecov Report

Merging #23 into master will increase coverage by 8.52%.
The diff coverage is 98.7%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #23      +/-   ##
==========================================
+ Coverage   43.93%   52.45%   +8.52%     
==========================================
  Files          12       13       +1     
  Lines         412      488      +76     
==========================================
+ Hits          181      256      +75     
- Misses        231      232       +1
Impacted Files Coverage Δ
grafana_api/grafana_face.py 100% <100%> (ø) ⬆️
grafana_api/api/__init__.py 100% <100%> (ø) ⬆️
grafana_api/api/team.py 98.64% <98.64%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 143511d...4bc0ebf. Read the comment docs.

Copy link
Owner

@m0nhawk m0nhawk left a comment

Choose a reason for hiding this comment

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

Thank you for adding Teams support!

@m0nhawk m0nhawk merged commit ca4d03a into m0nhawk:master May 5, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants