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

Loop detector refactoring #310

Merged
merged 11 commits into from
Apr 22, 2024
Merged

Loop detector refactoring #310

merged 11 commits into from
Apr 22, 2024

Conversation

eupp
Copy link
Collaborator

@eupp eupp commented Apr 19, 2024

Decouple LoopDetector from ManagedStrategy and extract it into separate file.

eupp and others added 10 commits April 19, 2024 13:32
* rename `LoopDetectorDecision` enum values
* make control-flow of `ManagedStrategy.newSwitchPoint` method more explicit

Signed-off-by: Evgeniy Moiseenko <[email protected]>
Signed-off-by: Evgeniy Moiseenko <[email protected]>
Signed-off-by: Evgeniy Moiseenko <[email protected]>
Signed-off-by: Evgeniy Moiseenko <[email protected]>
Signed-off-by: Evgeniy Moiseenko <[email protected]>
Signed-off-by: Evgeniy Moiseenko <[email protected]>
@eupp eupp requested a review from avpotapov00 April 19, 2024 21:23
Signed-off-by: Evgeniy Moiseenko <[email protected]>
@eupp eupp changed the base branch from master to develop April 19, 2024 21:42
@avpotapov00 avpotapov00 merged commit c1e38e7 into develop Apr 22, 2024
15 checks passed
@avpotapov00 avpotapov00 deleted the loop-detector-refactoring branch April 22, 2024 11:42
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