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

Added Word Wrap Problem #899

Merged
merged 10 commits into from
Apr 22, 2021
Merged

Conversation

ansi22-zz
Copy link
Contributor

Hey @tarun26091999
Kindly check this out, it will fix issue #427.

Copy link
Member

@tarun26091999 tarun26091999 left a comment

Choose a reason for hiding this comment

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

cannot have commits of 2 issues in 1 PR.

Copy link
Member

@tarun26091999 tarun26091999 left a comment

Choose a reason for hiding this comment

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

code is fine, just make the output easy to understand. Also link the code to appropriate folder.

@ansi22-zz
Copy link
Contributor Author

Hey, @tarun26091999 kindly check it, I have made the requested changes

@ansi22-zz
Copy link
Contributor Author

@tarun26091999 Sir Kindly review this.

Copy link
Member

@tarun26091999 tarun26091999 left a comment

Choose a reason for hiding this comment

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

give the output like this.
Line number 1: From word no. 1 to 3
Line number 2: From word no. 4 to 6
Line number 3: From word no. 7 to 10
Line number 4: From word no. 11 to 14
Line number 5: From word no. 15 to 17
Line number 6: From word no. 18 to 18

and also resolve the conflict.

@ansi22-zz
Copy link
Contributor Author

@tarun26091999 Sir am unable to understand the conflict here can you please let me know how to resolve it.

@tarun26091999 tarun26091999 merged commit 741e55f into Algo-Phantoms:main Apr 22, 2021
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