Skip to content
This repository was archived by the owner on Sep 5, 2020. It is now read-only.

Minor spelling/grammatical changes. #3083

Closed
wants to merge 1 commit into from
Closed

Minor spelling/grammatical changes. #3083

wants to merge 1 commit into from

Conversation

ericjarvies
Copy link
Contributor

No description provided.

Copy link
Member

@wolovim wolovim left a comment

Choose a reason for hiding this comment

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

Looks good. Small tweak requested.


***

### mist.menu.update(id, [, options] [, callback])

Works like `mist.menu.add()`, but all but the `id` parameters are optional.
Works like `mist.menu.add()`, but all of the `id` parameters are optional.
Copy link
Member

Choose a reason for hiding this comment

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

You've changed the meaning of this sentence. All but the ID param is optional. Agree that the sentence is clunky, though. Feel free to give it another whirl or revert this change.

Copy link
Contributor Author

@ericjarvies ericjarvies left a comment

Choose a reason for hiding this comment

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

Certainly less clunky now.

@wolovim wolovim mentioned this pull request Oct 3, 2017
@wolovim
Copy link
Member

wolovim commented Oct 3, 2017

Thanks for the PR. I've moved your changes into a new PR, where I've added a small change. (#3118)

@wolovim wolovim closed this Oct 3, 2017
@lock
Copy link

lock bot commented Mar 30, 2018

This thread has been automatically locked because it has not had recent activity. Please open a new issue for related bugs and link to relevant comments in this thread.

@lock lock bot locked and limited conversation to collaborators Mar 30, 2018
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