-
Notifications
You must be signed in to change notification settings - Fork 617
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 longest prefix suffix problem #783
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
code is fine. Add some more comments explaining your steps. Also space complexity is wrong. Give proper indentation and link it to the correct folder. remove line 9.
@tarun26091999 kindly check it out, I have made the changes. |
I WANT TO CONTRIBUTE |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you haven't linked it to the folder yet.
@tarun26091999 Sir, I have linked the file in string folder under branch issue-499, is it not showing there. Please tell how should I proceed. |
@tarun26091999 Sir kindly review this. |
@rish-singhal can you please review it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
resolve the conflict.
I have resolved the conflict @tarun26091999 kindly look into it. |
@ansi22 make correct indentations |
@tarun26091999 please once go through #899, scores for it didn't get updated 😅 |
Hey,
@anubhavitis @dheerajkotwani @tarun26091999 @todi-2000 @geekquad @plazzy99 @rudrakshi99
Kindly check this out it will fix issue #499 ,i.e, longest prefix suffix problem.
Thankyou!