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.

Sign up for GitHub

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

Jump to bottom

refactor #522

Open

krutik-kulkarni wants to merge 2 commits into chatty:master from krutik-kulkarni:master

krutik-kulkarni commented Mar 31, 2024

Done some refactoring, all the test cases are running after refactoring and project is building successfully.
Refactorings are as below:

Introduce explaining variable : Class: ColorCorrectionNew, Method: makeDarker
Extract Class: TwitchCommands class is handling the logic of mods which is moved by making a new class for
mods services.
Replace conditional with polymorphism: Class: DockUtils, DockSplit

kkulkarni and others added 2 commits

March 31, 2024 03:31


          refactor: all refactor methods covered

dbbb15f


          refactor: Mods class addition

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment