Merged
Conversation
Contributor
|
👋 Thank you for opening this pull request! We're excited to review your contribution. Please give us a moment, and we'll get back to you shortly! Feel free to join our community on Discord to discuss more! |
Contributor
Author
|
"Hi @UTSAVS26 / @pavitraag , I've created a PR and have been waiting for your review—please let me know if any changes are needed. Thanks!" |
Contributor
Author
|
@UTSAVS26 Please update my points at your earliest convenience |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
fixes: #494
This pull request includes several enhancements and refactoring of the
ListManipulatorclass in thepysnippets/list/list_manipulation.pyfile, as well as updates to the corresponding test cases. The main changes involve adding new utility methods for list manipulation and removing redundant docstrings.Enhancements and new utility methods:
Added
chunk_listmethod to split a list into smaller chunks of a given size.Added
most_frequent_elementmethod to identify the most frequently occurring element in a list.Added
rotate_listmethod to rotate the list by a given number of positions.Added
unique_elementsmethod to get elements that appear exactly once in the list.Added
find_pairs_with_summethod to find all pairs of numbers that sum to a specific target.Added test cases for the new methods
chunk_list,most_frequent_element,rotate_list,unique_elements, andfind_pairs_with_suminpysnippets/list/test_list_manipulation.py.All test cases have been executed successfully, confirming the correctness of the changes.