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

Deps/remove refmterr rely #37

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Deps/remove refmterr rely #37

wants to merge 7 commits into from

Conversation

Et7f3
Copy link
Member

@Et7f3 Et7f3 commented May 18, 2020

Hy @bryphe in response to revery-ui/revery#844
We pull rely with rench (that is used by revery). With this PR rely is no longer pulled from revery consumer.
Also the diff with hello-reason setup is we don't use the dsl #{self.name} but we use Rench this change is a breaking change so I haven't done.
I have checked lower bound of OCaml until 4.06 (no project use lower so it is fine).
While updating this PR I have seen we pull useless deps that aren't mentionned like lambda-term (only present in lockfile) or lwt (I presume because of threads.posix) I have pruned them. It will help for revery-ui/revery#890

@Et7f3 Et7f3 mentioned this pull request Jun 4, 2020
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant