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

undefined method `multiple?' for Grape::Validations::Types:Module #2221

Closed
cristofer opened this issue Dec 31, 2021 · 3 comments
Closed

undefined method `multiple?' for Grape::Validations::Types:Module #2221

cristofer opened this issue Dec 31, 2021 · 3 comments

Comments

@cristofer
Copy link

Hey guys,

build(deps): bump grape from 1.6.0 to 1.6.1

This PR #2209 introduced this check https://github.com/ruby-grape/grape/pull/2209/files#diff-e97e1900db45e8b2dbf58e1a31c80ca18d47fe7d7a2ae27a6c98a076c6936b18R177, which breaks in our CI:

/home/circleci/repo/vendor/bundle/ruby/2.6.0/gems/grape-1.6.1/lib/grape/validations/params_scope.rb:359:in `infer_coercion'
/home/circleci/repo/vendor/bundle/ruby/2.6.0/gems/grape-1.6.1/lib/grape/validations/params_scope.rb:274:in `validates'
/home/circleci/repo/vendor/bundle/ruby/2.6.0/gems/grape-1.6.1/lib/grape/validations/params_scope.rb:195:in `validate_attributes'
/home/circleci/repo/vendor/bundle/ruby/2.6.0/gems/grape-1.6.1/lib/grape/dsl/parameters.rb:135:in `requires'

I went through the loads specified here https://github.com/ruby-grape/grape/pull/2209/files#diff-b56aab6c58cef370018ab45d697b38247b363bf034afcd2e4b4d9fff05c3de0eR230 , and indeed couldnt find anything that could make a reference to multiple?

Please let me know if you need anything else in order to reproduce the problem.

Thank you for your great work!

@cristofer
Copy link
Author

NVM, I just saw the fix here #2219, thanks for the quick reaction to that

@dblock
Copy link
Member

dblock commented Dec 31, 2021

@dm1try should we yank 1.6.1 from rubygems? is everybody running into this?

@dm1try
Copy link
Member

dm1try commented Jan 1, 2022

@dblock maybe, it's a new experience :)
though, the upgrade to 1.6.2 should work for most users.

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

3 participants