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

interfaces: fix typo "daemonNotify" (add missing "n") #5499

Merged
merged 1 commit into from Jul 12, 2018

Conversation

zyga
Copy link
Collaborator

@zyga zyga commented Jul 12, 2018

This was an easy miss since the first word ends with n and the second word starts with n.

Signed-off-by: Zygmunt Krynicki zygmunt.krynicki@canonical.com

Signed-off-by: Zygmunt Krynicki <zygmunt.krynicki@canonical.com>
@zyga zyga added the Simple 😃 A small PR which can be reviewed quickly label Jul 12, 2018
@codecov-io
Copy link

Codecov Report

Merging #5499 into master will decrease coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #5499      +/-   ##
==========================================
- Coverage   78.96%   78.95%   -0.01%     
==========================================
  Files         512      512              
  Lines       38734    38734              
==========================================
- Hits        30586    30584       -2     
- Misses       5692     5693       +1     
- Partials     2456     2457       +1
Impacted Files Coverage Δ
interfaces/builtin/daemon_notify.go 93.75% <100%> (ø) ⬆️
overlord/hookstate/hookmgr.go 74.27% <0%> (-0.98%) ⬇️

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 14aef09...c3c194f. Read the comment docs.

@zyga zyga merged commit 382d364 into snapcore:master Jul 12, 2018
@zyga zyga deleted the tweak/daemon-notify-typo branch July 12, 2018 20:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Simple 😃 A small PR which can be reviewed quickly
Projects
None yet
3 participants