Open dajuguan opened 4 days ago
The description is too short for such a large change set IMO, could you elaborate in more detail so that the review can be easier?
The description is too short for such a large change set IMO, could you elaborate in more detail so that the review can be easier?
Considering the extensive context, it might be better to split the changes into multiple PRs during a future rebase. For now, I suggest focusing on reviewing cosmetic aspects like naming and variable conventions, as that seems more feasible.
Description:
op-challenger2/game/fault/solver/solver_test.go
andop-challenger2/game/fault/solver/game_solver_test.go
Tests
op-challenger2: all tests passed
alphabet game E2E: all tests passed