search

samtay / tetris

A terminal interface for Tetris
Other
871 stars 40 forks source link

Update Game.hs #11

Closed bohadi closed 3 years ago

bohadi commented 3 years ago

0.47 API changes: Changed Brick.Main.customMain so that it now takes an additional (first) argument: the initial Vty handle to use. This lets the caller have more control over the terminal state when, for example, they have previously set up Vty to do other work before calling customMain.

via brick/CHANGELOG.md

samtay commented 3 years ago

Hi @bohadi. This game was built using a stack LTS (currently lts-13.0 and brick-0.45 - see stack.yaml). If you want to update the code to use brick-0.47, you'll have to update that file to make sure everything else builds properly with 0.47 (and to make sure that, when other people try to install this code, they do so against the correct version of brick). You can test out your changes with, e.g. stack build --pedantic.

However, if you're going to bump the lib, I would suggest trying to bump the entire stack lts and get onto a much later version of brick (I see it's up to brick-0.58.1 on lts-17.5). I'll happily merge a PR that helps keep the code up to date!

Thanks.