tkshill/Quarto

Remove the NoOp message type from Game.elm

Closed this issue · 2 comments

tkshill commented

Issue Context

In the Game.elm file, we've defined a NoOp Msg that we don't seem to actually need. For the sake of reducing the complexity of the code, it should be removed.

Suggested Solution

  • Remove the NoOp from the acceptable Msg cases in Game.elm
  • Remove the corresponding reference to it in the update function of Game.elm

Alternatives Considered

N/A

Additional Resources

You can take a peek at the file here before you start if you like.

See here in the read me for how to run and install the application.

See here in contributing for the basics on forking and cloning the repository.

github-actions commented

Thank you for the issue! We'll take your request into consideration and will follow up soon if we decide to tackle this issue.
To all contributors: any issue that is labeled Status: Available is up for grabs.
To claim an issue to work on, please leave a comment. If you've claimed the issue and need help, please leave a comment letting us know and we can schedule some time to help.
For full info on how to contribute, please check out our contributors guide.

Leoactionz commented

@tkshill, I would love to grab this as well. Any helpful resources will be appreciated