Skip to content

Removed colon ambiguity#17

Merged
rimi-itk merged 2 commits intodevelopfrom
feature/consistent_use_of_colon
Mar 22, 2019
Merged

Removed colon ambiguity#17
rimi-itk merged 2 commits intodevelopfrom
feature/consistent_use_of_colon

Conversation

@tuj
Copy link
Copy Markdown
Contributor

@tuj tuj commented Mar 22, 2019

No description provided.

@tuj tuj requested review from cableman and rimi-itk March 22, 2019 12:20
Copy link
Copy Markdown
Contributor

@rimi-itk rimi-itk left a comment

Choose a reason for hiding this comment

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

Nice. For (even more) consistency, we should consider adding a linebreak after each command in the help section and settle on a fixed indentation of the documentation block, e.g.

…

  url
        Print url …

  open
        Open url …
…

Copy link
Copy Markdown
Contributor

@rimi-itk rimi-itk left a comment

Choose a reason for hiding this comment

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

LGTM.

@rimi-itk rimi-itk merged commit db4fd76 into develop Mar 22, 2019
@rimi-itk rimi-itk deleted the feature/consistent_use_of_colon branch March 22, 2019 14:46
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.

2 participants