search

yawaramin / re-web

Experimental web framework for ReasonML & OCaml
https://yawaramin.github.io/re-web/re-web/index.html
MIT License
261 stars 8 forks source link

update some deps #40

Closed idkjs closed 3 years ago

idkjs commented 3 years ago

I updated the deps to get rid of some errors. Some of the tests arent working though. Here it is if you want it.

yawaramin commented 3 years ago

Hi Alain, the main issue here is that I like to keep 'semantic' version bounds on my dependencies. If there has been a major version update of any of them, I will like to know about it.

Secondary issue is that I am intentionally not generating an opam file in this project ( #34 ).

Tertiary issue is the commented-out tests will need to be fixed :-)

idkjs commented 3 years ago

@yawaramin I figured as much, hence the draft. Because I love and respect what you do, I just want this thing to work when someone adds it to a project. This is how I got to work.

If understand the semantics comment, how would I get esy to install list the actual deps in package.json?

For the tests I need to look at it some more. I could not seem to figure it out right off looking at it.

I clearly do not need this. Just wanted to work without the errors.

yawaramin commented 3 years ago

No prob. Just open an issue with the errors you're getting, what version you're using, etc., and we can take a look at it.