Refactoring and Structural Improvements#1
Merged
Conversation
Removed unnecessary comments, resolved errors and warnings, improved code readability with added spacing, and implemented flag checks for validation. Additional validations after next update.
Extensive refactoring, improved comments, optimized code structure, introduced flags, and resolved errors for better functionality GitIgnore additionally added for .exe extension and output.bmp file
Additionally removed .vcode, personal included setup files separate from each developer
Moved main files into separate folder while retaining Readme and license in root directory.
VerzatileDev
commented
Aug 18, 2023
VerzatileDev
left a comment
Contributor
Author
There was a problem hiding this comment.
Everything seems to be as intended, except folder structure has been changed which might bring difficulties for other developers. Note to Stash Changes before Pulling and check where nessecary.
Sommos
reviewed
Aug 18, 2023
Sommos
left a comment
Contributor
There was a problem hiding this comment.
Discussed previously, works with no merge conflicts. Merging.
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.
This pull request introduces a series of improvements to the codebase, focusing on enhancing readability, organization, and overall functionality.
Reorganized Folder Structure: The main files have been moved into their respective separate folder, creating a more organized project structure. The Readme and license files remain in the root directory of the repository.
Enhanced Codebase: Extensive refactoring has been carried out to improve the codebase. Comments have been optimized for clarity, unnecessary comments have been removed, and code placement has been carefully structured. Additionally, flag checks have been introduced for validation purposes.
Refactored Codebase: In previous commits, unnecessary comments were removed, errors and warnings were resolved, and code readability was improved by adding proper spacing. Flag checks were also implemented to ensure validation in specific areas.