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

Lots of warnings w/r/t Schema fn calls #105

Closed
venantius opened this issue Sep 12, 2015 · 4 comments
Closed

Lots of warnings w/r/t Schema fn calls #105

venantius opened this issue Sep 12, 2015 · 4 comments

Comments

@venantius
Copy link

I had plumbing pulled into my project by om-tools. I'm also using the newest version of Schema for my backend validation. I believe the combination of the two is what's resulting in quite a few warnings when I boot up Figwheel (see below).

william_jarvis@jackalope ~/C/shrike> lein figwheel
Figwheel: Starting server at http://localhost:3449
Focusing on build ids: app
Compiling "resources/public/js/app.js" from ["src/cljs"]...
WARNING: schema.core/either is deprecated. at line 23 file:/Users/william_jarvis/.m2/repository/prismatic/plumbing/0.4.4/plumbing-0.4.4.jar!/plumbing/fnk/schema.cljs
WARNING: schema.core/either is deprecated. at line 27 file:/Users/william_jarvis/.m2/repository/prismatic/plumbing/0.4.4/plumbing-0.4.4.jar!/plumbing/fnk/schema.cljs
WARNING: schema.core/both is deprecated. at line 66 file:/Users/william_jarvis/.m2/repository/prismatic/plumbing/0.4.4/plumbing-0.4.4.jar!/plumbing/fnk/schema.cljs
WARNING: schema.core/either is deprecated. at line 23 resources/public/js/out/plumbing/fnk/schema.cljs
WARNING: schema.core/either is deprecated. at line 27 resources/public/js/out/plumbing/fnk/schema.cljs
WARNING: schema.core/both is deprecated. at line 66 resources/public/js/out/plumbing/fnk/schema.cljs
Successfully compiled "resources/public/js/app.js" in 10.943 seconds.

Any chance of bumping Plumbing's dependency on Schema to a more recent version and cleaning up the deprecation dependencies?

@w01fe
Copy link
Member

w01fe commented Sep 12, 2015

Oops, didn't realize we were using both in plumbing. Will do.

@w01fe
Copy link
Member

w01fe commented Sep 12, 2015

Just released 0.5.0 which should fix this -- let me know if it works for you. Thanks for the report!

@w01fe w01fe closed this as completed Sep 12, 2015
@venantius
Copy link
Author

Thanks for the hasty response! :)

@venantius
Copy link
Author

Might want to make a note to update om-tools to update it's dependency on plumbing, too.

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

No branches or pull requests

2 participants