Skip to content

Conversation

@Tishasoumya-02
Copy link
Contributor

@Tishasoumya-02 Tishasoumya-02 commented Sep 6, 2021

Description of Change

Added an algorithm for Reversing a single linked list.

Checklist

  • Added description of change
  • Added file name matches File name guidelines
  • Added documentation so that the program is self-explanatory and educational - Doxygen guidelines
  • Relevant documentation/comments is changed or added
  • PR title follows semantic commit guidelines
  • Search previous suggestions before making a new one, as yours may be a duplicate.
  • I acknowledge that all my contributions will be made under the project's license.

Notes:
Kindly review my PR and let me know any corrections.

G3N3R4LE4RL
G3N3R4LE4RL previously approved these changes Sep 6, 2021
iamrajiv
iamrajiv previously approved these changes Sep 6, 2021
Copy link
Member

@mishraabhinn mishraabhinn left a comment

Choose a reason for hiding this comment

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

HI. Thanks for creating pr @Tishasoumya-02 ,But this pr is not upto repository standards. Plz look to contribution guidelines

@Panquesito7 Panquesito7 changed the title add: reverse linked list algorithim feat: Add reverse linked list algorithim Sep 6, 2021
@Panquesito7 Panquesito7 added awaiting modification Do not merge until modifications are made enhancement New feature or request requested changes changes have been requested labels Sep 6, 2021
@Panquesito7 Panquesito7 added the Proper Documentation Required requested to write the documentation properly label Sep 6, 2021
Co-authored-by: David Leal <halfpacho@gmail.com>
@Tishasoumya-02 Tishasoumya-02 dismissed stale reviews from iamrajiv and G3N3R4LE4RL via 55de9f9 September 6, 2021 18:28
Co-authored-by: David Leal <halfpacho@gmail.com>
Co-authored-by: David Leal <halfpacho@gmail.com>
Copy link
Member

@Panquesito7 Panquesito7 left a comment

Choose a reason for hiding this comment

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

Looks like there was a merge conflict not solved well here. 👀

Copy link
Member

@ayaankhan98 ayaankhan98 left a comment

Choose a reason for hiding this comment

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

@Tishasoumya-02 please mark the suggestions as resolved if you have incorporated them.

Co-authored-by: David Leal <halfpacho@gmail.com>
Copy link
Member

@Panquesito7 Panquesito7 left a comment

Choose a reason for hiding this comment

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

Awesome work, @Tishasoumya-02! It's been great for your first contribution here! We hope you keep contributing! Thank you! 😄👍🎉

Copy link
Member

@aminoxix aminoxix left a comment

Choose a reason for hiding this comment

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

@Tishasoumya-02: LGTM! 🥳

@Panquesito7 Panquesito7 merged commit f7a5aec into TheAlgorithms:master Sep 27, 2021
@Panquesito7
Copy link
Member

Thanks for reviewing @mishraabhinn and @Amino19! 🙂

@Panquesito7 Panquesito7 changed the title feat: Add reverse linked list algorithim feat: Add reverse linked list algorithm Sep 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Approved; waiting for merge enhancement New feature or request

7 participants