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

fix message #16

Merged
merged 3 commits into from
Sep 9, 2016
Merged

fix message #16

merged 3 commits into from
Sep 9, 2016

Conversation

ngawor
Copy link
Contributor

@ngawor ngawor commented Sep 9, 2016

No description provided.

@ngawor ngawor assigned ngawor and chambrid and unassigned ngawor Sep 9, 2016
@@ -15,7 +15,7 @@
"redis.slowlog.timestamp": "Timestamp",
"redis.slowlog.execution.time": "Microseconds for Execution",
"redis.slowlog.command": "Command",
"redis.slowlog.not.found": "No commands were found in the slow logs for the given threshold of %s microseconds.",
"redis.slowlog.not.found": "No commands were found in the slow logs for the given threshold of %s commands.",
Copy link
Contributor

Choose a reason for hiding this comment

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

I would just remove the "for the given threshold part of the sentence."

@coveralls
Copy link

Coverage Status

Coverage remained the same at 79.435% when pulling cd4b62d on fix_message into ade19ed on master.

@chambrid
Copy link
Contributor

chambrid commented Sep 9, 2016

👍

@coveralls
Copy link

Coverage Status

Coverage remained the same at 79.435% when pulling b02ee5a on fix_message into ade19ed on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 79.435% when pulling b02ee5a on fix_message into ade19ed on master.

@chambrid chambrid merged commit 35de372 into master Sep 9, 2016
@chambrid chambrid deleted the fix_message branch September 9, 2016 18:49
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.

None yet

3 participants