Skip to content

Conversation

amitd291
Copy link
Contributor

No description provided.

@rastdeepanshu
Copy link
Contributor

👍

@npathai npathai merged commit 2f33094 into iluwatar:master Sep 27, 2017
@npathai
Copy link
Contributor

npathai commented Sep 27, 2017

@amitdash291 Thanks for contribution. It is merged. Though I feel that keeping times(1) makes things more explicit and readable.

@amitd291
Copy link
Contributor Author

@npathai for readability it made sense to not remove it, though, on the other hand, most of the devs implicitly read times(1) when it isn't present

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants