String concatenation is replaced with append() chains. #60

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.

Sign up for GitHub

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

Jump to bottom

Closed

kamaci wants to merge 1 commit into ipfs-shipyard:master from kamaci:master_4

Contributor

kamaci commented Mar 6, 2018

String concatenation is replaced with append() chains.


          String concatenation is replaced with append() chains.

b071a4c

Collaborator

ianopolous commented Mar 6, 2018

I prefer keeping the logically grouped things together in a single append call.

I also prefer .* imports than specific class imports.

ianopolous closed this

Contributor Author

kamaci commented Mar 6, 2018

@ianopolous I've created #61.

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

Labels

None yet