Skip to content

fix typo the description of interpreter menu.#1037

Closed
astroshim wants to merge 2 commits into
apache:masterfrom
astroshim:feat/typoInterpreters
Closed

fix typo the description of interpreter menu.#1037
astroshim wants to merge 2 commits into
apache:masterfrom
astroshim:feat/typoInterpreters

Conversation

@astroshim
Copy link
Copy Markdown
Contributor

What is this PR for?

This PR is for fixing typo of the interpreter menu description.

What type of PR is it?

Bug Fix

Screenshots (if appropriate)

  • before
    image
  • after
    image

Questions:

  • Does the licenses files need update? no
  • Is there breaking changes for older versions? no
  • Does this needs documentation? no

@minahlee
Copy link
Copy Markdown
Member

minahlee commented Jun 18, 2016

Nice catch! Can we make space to be consistent between create / edit / remove and bind/unbind?

@astroshim
Copy link
Copy Markdown
Contributor Author

@minahlee Thanks for pointing out. I just fixed.

@felixcheung
Copy link
Copy Markdown
Member

LGTM

asfgit pushed a commit that referenced this pull request Jun 20, 2016
### What is this PR for?
This PR is for fixing typo of the interpreter menu description.

### What type of PR is it?
Bug Fix

### Screenshots (if appropriate)
  - before
 ![image](https://cloud.githubusercontent.com/assets/3348133/16156875/1dba142e-34f1-11e6-937b-c3d2c75a6c42.png)

  - after
![image](https://cloud.githubusercontent.com/assets/3348133/16156901/365979a2-34f1-11e6-91a7-4c99b78a1d86.png)

### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? no

Author: astroshim <hsshim@nflabs.com>

Closes #1037 from astroshim/feat/typoInterpreters and squashes the following commits:

97f1fb7 [astroshim] space bind/unbind too.
64f1f60 [astroshim] fix typo description of interpreter menu.

(cherry picked from commit 2054f9f)
Signed-off-by: Mina Lee <minalee@apache.org>
@asfgit asfgit closed this in 2054f9f Jun 20, 2016
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.

3 participants