Skip to content

Conversation

Sergey-Starushkin
Copy link

No description provided.

@vramaniuk vramaniuk self-requested a review August 25, 2020 09:54
Copy link
Collaborator

@vramaniuk vramaniuk left a comment

Choose a reason for hiding this comment

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

I'll mark the PR as "Draft", please click "ready for review" when it will be finished. Thank you!

let i = 0, result = "";

while (i < lines[0].length) {
let s = lines[0].substr(i, 3) + lines[1].substr(i, 3) + lines[2].substr(i, 3);
Copy link
Collaborator

Choose a reason for hiding this comment

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

method String.prototype.substr is deprecated
Please, use String.prototype.substring

Comment on lines +126 to +127
yield text.substr(0, i);
text = text.substr(i + 1);
Copy link
Collaborator

Choose a reason for hiding this comment

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

method String.prototype.substr is deprecated
Please, use String.prototype.substring

@vramaniuk
Copy link
Collaborator

Please, privide the direct link to your task success travis build

@vramaniuk vramaniuk marked this pull request as draft August 25, 2020 09:58
@Sergey-Starushkin
Copy link
Author

@Sergey-Starushkin Sergey-Starushkin marked this pull request as ready for review August 31, 2020 19:57
@vramaniuk
Copy link
Collaborator

Dirty PR. Please, see notes for tasks 1-4

And implement previous notes (substr)

I'll mark the PR as "Draft", please click "ready for review" when it will be finished. Thank you!

@vramaniuk vramaniuk marked this pull request as draft September 1, 2020 15:38
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