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

Support (and require) brick 1.0 #1919

Merged
merged 4 commits into from Aug 17, 2022
Merged

Support (and require) brick 1.0 #1919

merged 4 commits into from Aug 17, 2022

Conversation

simonmichael
Copy link
Owner

@simonmichael simonmichael commented Aug 17, 2022

Adapt to brick's new API, #1889.

  • dev: make ghci[d]-ui: fix/update
  • pkg: ui: use/require brick 1.0+ (1889)
  • stack: use brick 1.0 with older GHCs also

@crocodile-code-review
Copy link

Review on Crocodile

@simonmichael simonmichael changed the title brick 1 Support (and require) brick 1.0 Aug 17, 2022
@simonmichael simonmichael merged commit fcdc07b into master Aug 17, 2022
@simonmichael simonmichael deleted the brick-1 branch August 17, 2022 14:57
simonmichael added a commit that referenced this pull request Sep 7, 2022
Follow-on from the brick 1.0 migration work (#1889, #1919).

These new types aim to be more restrictive, allowing fewer invalid
states, and easier to inspect and debug. The screen types store only
state, not behaviour (functions), and there is no longer a circular
dependency between UIState and Screen.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ui The hledger-ui tool.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant