-
Notifications
You must be signed in to change notification settings - Fork 294
Conversation
Updated glossary terminology and hyperlinks using snapctl and snapd to snaptel and snapteld, respectively.
@@ -71,10 +71,10 @@ Snap is simple in scope and it becomes more simple when you know the terminology | |||
### Snap Telemetry | |||
* The full name of the Snap project, used mostly for easy searching (like snap-telemetry.io) or hashtag (#SnapTelemetry) | |||
|
|||
### `snapctl` | |||
### snaptel |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you intentionally remove grave accent?
* The command-line interface (CLI) for Snap, released as a [binary][binary] | ||
|
||
### `snapd` | ||
### snapteld |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same question
Can we make this one commit? Other than that LGTM. |
@IRCody as maintainers, we can do that ourselves as of the latest GitHub release 👍 |
@mjbrender: Doesn't that result in not having a merge commit? |
@IRCody: good call - I was wrong. If we're staying consistent with merge commits, we shouldn't use that feature 😞 cc @kindermoumoute (Didn't mean to tag you @obourdon) |
Is it really bad not to have this merge commit? |
@kindermoumoute it's not a big deal to miss it this time, but we have stuck to using them and should remain consistent. |
Summary of changes:
Testing done:
@intelsdi-x/snap-maintainers