Skip to content
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

Refactoring #1502

Closed
wants to merge 3 commits into from
Closed

Refactoring #1502

wants to merge 3 commits into from

Conversation

AlexanderStocks
Copy link

Description:

As a part of my dissertation work, I've implemented a series of automated refactorings and formatting to improve the overall code quality of this repository:

Reformat: By adhering to Google's Java Code Style guidelines, I've reformatted 408 files. This step aims to enhance code readability and ensure consistency with widely accepted programming style standards.

CollapseNestedIfStatements: I have executed the CollapseNestedIfStatements refactoring on 8 files. This refactoring effort is aimed at simplifying the logic of the code and making it more readable.

RemoveDuplication: I have executed the RemoveDuplication refactoring on 18 file, aiming to streamline the codebase by eliminating redundant code blocks.

These changes were all performed with the aid of the Refactoring Janitor tool.

Request for Feedback:

As this effort is part of my dissertation project, your input and feedback on the changes I've made are greatly appreciated. Your insights will significantly contribute to my research and learning process.

@spencergibb
Copy link
Contributor

I don't think this is helpful. Especially when a project has a different set of code style standards than Google

@AlexanderStocks AlexanderStocks closed this by deleting the head repository Aug 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants