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.

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

Cleanup #13

Closed
Dachgruber opened this issue Jan 26, 2022 · 1 comment
Closed

Cleanup #13

Dachgruber opened this issue Jan 26, 2022 · 1 comment
Assignees
@Dachgruber @baschi29
Labels
controller documentation Improvements or additions to documentation model features regarding the model view
Milestone
Meilenstein 4

Comments

@Dachgruber
Copy link
Collaborator

Major cleanup for the code is needed! Javadoc-comments, deletion of unused classes/code and maybe some rework of some old code in the model/controller class
@Dachgruber Dachgruber added model features regarding the model controller view labels Jan 26, 2022
@Dachgruber Dachgruber added this to the Meilenstein 4 milestone Jan 26, 2022
@baschi29 baschi29 added the documentation Improvements or additions to documentation label Jan 26, 2022
baschi29 added a commit that referenced this issue Jan 26, 2022
@baschi29
Removed obsolete getOldPos() method, #13
782bc43
baschi29 added a commit that referenced this issue Jan 26, 2022
@baschi29
Comments and Cleanup, #13
ddd3904
baschi29 added a commit that referenced this issue Jan 27, 2022
@baschi29
Comments and Cleanup #13
6e18e81
@baschi29 baschi29 mentioned this issue Jan 31, 2022
Closed
Dachgruber added a commit that referenced this issue Feb 1, 2022
@Dachgruber
updated javadoc
654674e 
updated the javadoc comments for better display, see #13
baschi29 added a commit that referenced this issue Feb 1, 2022
@baschi29
consistent JavaDoc authors #13
2ec4bad
baschi29 added a commit that referenced this issue Feb 1, 2022
@baschi29
Controller and partly model cleanup and javadoc #13
c8834b2
@baschi29 baschi29 pinned this issue Feb 1, 2022
baschi29 added a commit that referenced this issue Feb 3, 2022
@baschi29
Model comments #13
0e5e79b
baschi29 added a commit that referenced this issue Feb 3, 2022
@baschi29
renaming various model variables #13
24969e6
baschi29 added a commit that referenced this issue Feb 3, 2022
@baschi29
more renaming #13
00e3be0
baschi29 added a commit that referenced this issue Feb 3, 2022
@baschi29
Interface comments #13
f80dfd4
baschi29 added a commit that referenced this issue Feb 3, 2022
@baschi29
Removed todos #13
bffdc1d
@baschi29
Copy link
Owner

baschi29 commented Feb 3, 2022

Comments should be done on my side, there is still room for cleaning up the code - will probably will not do this before next week

baschi29 added a commit that referenced this issue Feb 5, 2022
@baschi29
Cleanup for #13 but needs comments
cb0e989
baschi29 added a commit that referenced this issue Feb 5, 2022
@baschi29
Comments #13
0e4c44e
Dachgruber added a commit that referenced this issue Feb 6, 2022
@Dachgruber
removed unused imports, comments and added SuppressedWarnings #13
a0d4658
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Dachgruber Dachgruber

@baschi29 baschi29

Labels
controller documentation Improvements or additions to documentation model features regarding the model view
Projects
None yet
Milestone
Meilenstein 4
Development

No branches or pull requests
2 participants
@Dachgruber @baschi29