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

new dry-v syntax **DO NOT MERGE** #68

Closed
wants to merge 1 commit into from
Closed

Conversation

vasspilka
Copy link

Still not working after new syntax... getting

undefined method `filled' for [:val, [:predicate, [:key?, [:_csrf_token]]]]:Dry::Validation::Schema::Rule (NoMethodError)

any ideas??

Gem versions after bundle install

Resolving dependencies...
Using rake 11.2.2
Using bundler 1.11.2
Using byebug 9.0.5
Using concurrent-ruby 1.0.2
Using json 1.8.3
Using docile 1.1.5
Using simplecov-html 0.10.0
Using tins 1.6.0
Using thor 0.19.1
Using dry-equalizer 0.2.0
Using dry-monads 0.0.1
Using inflecto 0.0.2
Using hanami-utils 0.8.0 from git://github.com/hanami/utils.git (at 0.8.x@8fcd94b)
Using rack 1.6.4
Using tilt 2.0.5
Using minitest 5.9.0
Using yard 0.8.7.6
Using dry-configurable 0.1.6
Using simplecov 0.11.2
Using term-ansicolor 1.3.2
Using hanami-helpers 0.4.0 from source at `.`
Using hanami-controller 0.7.0 from git://github.com/hanami/controller.git (at 0.7.x@0f3a22b)
Using hanami-view 0.7.0 from git://github.com/hanami/view.git (at 0.7.x@6a51e5c)
Using dry-container 0.3.4
Using coveralls 0.8.13
Using dry-logic 0.2.3
Using dry-types 0.7.2
Using dry-validation 0.7.4
Using hanami-validations 0.6.0 from git://github.com/hanami/validations.git (at 0.6.x@a448067)

@vasspilka
Copy link
Author

should fix #67

@coveralls
Copy link

coveralls commented Jun 12, 2016

Coverage Status

Coverage decreased (-53.5%) to 42.338% when pulling 02941d9 on vasspilka:dry-v into d3c2f6f on hanami:master.

@jodosha
Copy link
Member

jodosha commented Jun 20, 2016

@vasspilka Thanks for this PR, but it wasn't enough. I fixed it with b534567

@jodosha jodosha closed this Jun 20, 2016
@jodosha jodosha self-assigned this Jun 20, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants