Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

putText and putTextLn #81

Closed
vrom911 opened this issue Nov 13, 2017 · 5 comments
Closed

putText and putTextLn #81

vrom911 opened this issue Nov 13, 2017 · 5 comments
Assignees

Comments

@vrom911
Copy link
Contributor

vrom911 commented Nov 13, 2017

For the moment there is only putText function in universum which behaves like putStrLn.
I think it would be convenient to have putText which would be specialized version of putStr and putTextLn — version of putStrLn.

@chshersh
Copy link
Contributor

@vrom911 I agree! This should have been done long ago... But I think there're a lot of usages of putText currently and a lot of code can become broken 😞 I don't see how we can perform migration to save old behavior and also have consistent naming. But I think we really should have putText and putTextLn, because current naming is far from perfect 😭

@vrom911
Copy link
Contributor Author

vrom911 commented Nov 13, 2017

@chshersh, it's really sad..(
Probably we could name the new one as putTextInline then, so there won't be any conflicts? Or any other more suitable alternative?

@gromakovsky
Copy link
Member

universum-1.0.0 😉

@chshersh
Copy link
Contributor

@gromakovsky I want #5 be done before releasing universum-1.0.0.. And probably something else before we will announce universum.

@chshersh chshersh added this to the v1.0.0 milestone Nov 13, 2017
@chshersh
Copy link
Contributor

So the solution is:

  1. Rename putText to putTextLn
  2. Add putText whish is equsl to putStr.

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

No branches or pull requests

3 participants