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

Refactor: Eff #2

Open
BlastWind opened this issue Dec 15, 2024 · 1 comment
Open

Refactor: Eff #2

BlastWind opened this issue Dec 15, 2024 · 1 comment
Milestone
Eff, Lenses, Card...

Comments

@BlastWind
Copy link
Owner

No description provided.

BlastWind added a commit that referenced this issue Dec 16, 2024
@BlastWind
Completes #2. Defined random effect; replaced all traces of MonadRand…
83cdc67 
…om with the new effect!
@BlastWind
Copy link
Owner Author

Actually, the repo should definitely have the following in place before tackling #3 and #4:

  1. Refactor many functions involving PlayerState to GameState.
  2. Refactor most things involving GameState -> GameState to a (State GameState :> es) => Eff es a.
  3. Make GameState include a Map UUID PlayerState and not just two PlayerStates as it is right now.
  4. Zoom appropriately. Note, zoom worked with Zoom (which depended on mtl-style MonadState), so there probably will be some rewritings for zoom to work with the state effect.

@BlastWind BlastWind reopened this Dec 16, 2024
BlastWind added a commit that referenced this issue Jan 12, 2025
@BlastWind
More State pattern refactor; PlayerMap; More Eff integration #2
60e5c3c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
Eff, Lenses, Card DSL, Test Handlers
Development

No branches or pull requests
1 participant
@BlastWind