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.

Sign up for GitHub

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

Jump to bottom

Adding Timeticks unit #11183

Merged
merged 1 commit into from
Mar 12, 2018
Merged

Adding Timeticks unit #11183

merged 1 commit into from
Mar 12, 2018

Conversation

jtyr
Copy link
Contributor

@jtyr jtyr commented Mar 9, 2018

This PR is adding a new time unit called Timeticks which is defined like "hundredth of a second" and is common in the world of SNMP monitoring.

@CLAassistant
Copy link

CLAassistant commented Mar 9, 2018 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@codecov-io
Copy link

codecov-io commented Mar 9, 2018 •
edited
Loading

Codecov Report

Merging #11183 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #11183      +/-   ##
==========================================
- Coverage   51.87%   51.87%   -0.01%     
==========================================
  Files         349      349              
  Lines       24880    24882       +2     
  Branches     1422     1422              
==========================================
+ Hits        12907    12908       +1     
- Misses      11247    11248       +1     
  Partials      726      726

@bergquist bergquist merged commit 74602b1 into grafana:master Mar 12, 2018
@bergquist
Copy link
Contributor

Thank you!

@bergquist bergquist added this to the 5.1 milestone Mar 12, 2018
marefr added a commit that referenced this pull request Apr 16, 2018
@marefr
changlelog: notes about closing issues/pr's
645658d 
#11053, #11252, #10836, #11185, #11168, #11332, #11391, #11073, #9342, #11001, #11183, #11211, #11384, #11095, #10792, #11138, 
#11516
[skip ci]
@ying-jeanne ying-jeanne added the pr/external This PR is from external contributor label Apr 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bergquist bergquist bergquist approved these changes

Assignees
No one assigned
Labels
pr/external This PR is from external contributor
Projects
None yet
Milestone
5.1-beta1
Development

Successfully merging this pull request may close these issues.
5 participants
@jtyr @CLAassistant @codecov-io @bergquist @ying-jeanne