-
Notifications
You must be signed in to change notification settings - Fork 18.7k
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
[OTHER] Cleanup classes with very strange design #5164
Comments
Hey, I am working on "ArrayCombination" |
Hello I´m new here and I would like to help !!! |
@JeysiStrokk would you like to work on |
@vil02 I´ll gladly do it. I´ll try my best !! Thanks a lot !! |
I´m working on "LetterCombinationsOfPhoneNumber" |
I´m working on "StringMatchFiniteAutomata" |
I´m working on "ColumnarTranspositionCipher" |
Sorry, I refactored this class without seeing that you had already started. Now I saw in your repository that you only added the comments in the class. That's why I make the pull request. |
I´m working on "Combination" |
What would you like to share?
There are probably many classes in this repository, which have some strange design - we should update all of them. I think we can start with cleaning up these:
List of the classes needing redesign:
AllPathsFromSourceToTarget
ArrayCombination
(refactor: redesignArrayCombination
#5181 by @yuvashreenarayanan3)Combination
ColumnarTranspositionCipher
StringMatchFiniteAutomata
(Refactor: redesign StringMatchFiniteAutomata #5222 by @samuelfac)LetterCombinationsOfPhoneNumber
(Refactor: redesign LetterCombinationsOfPhoneNumber #5221 by @samuelfac)(@siriak, @BamaCharanChhandogi feel free to edit the list above)
Additional information
How to contribute
XYZ
", whereXYZ
is your selected class,cleanup_XYZ
,Also feel free to suggest a class, which should be added to the list above.
The text was updated successfully, but these errors were encountered: