Skip to content

Sprint 6 solution#3

Merged
ksupyl merged 20 commits into
mainfrom
sprint_6-solution
Mar 19, 2026
Merged

Sprint 6 solution#3
ksupyl merged 20 commits into
mainfrom
sprint_6-solution

Conversation

@ksupyl
Copy link
Copy Markdown
Owner

@ksupyl ksupyl commented Mar 18, 2026

No description provided.

ksupyl added 19 commits March 17, 2026 21:14
add remove method and linked list implementation
remove tasks from history on delete
duplicates, unlimited size, remove from start/middle/end
history cleanup on delete task/epic/subtask
to prevent external mutation, add setter test
now remove tasks from history;
add clear history tests;
fix getSubtask/getEpic to return copies
update Main to match sprint 6 scenario
fix subtaskCannotBeEpic - properly verify setId guard against id == epicId
add clearSubtasks history removal test
add epic immutability test after adding to manager
rename local variable in getEpicSubtasks to avoid shadowing class field
implement history with LinkedList and HashMap
encapsulate Node fields, optimize history add method
history now stores snapshots to prevent mutation via setters
fix CheckStyle violations in Node.java
fix CheckStyle violations in Managers.java
fix CheckStyle violations in HistoryManager.java
fix CheckStyle violations in Main.java
Comment thread src/service/InMemoryHistoryManager.java Outdated
eliminate duplicate tail assignment in linkLast
@ksupyl ksupyl merged commit 6fa1ab2 into main Mar 19, 2026
1 check passed
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