-
Notifications
You must be signed in to change notification settings - Fork 54
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
Cookstyle not detecting some bad Ruby Syntax #733
Comments
Thanks @kmf - I moved this issue over to the Cookstyle repo |
I noticed this the other day. Something changed in Rubocop itself and I need to track that down. |
tas50
added a commit
to chef/chef-workstation
that referenced
this issue
Sep 3, 2020
Cookstyle was no longer detecting bad ruby syntax due to a change (fix?) in the RuboCop engine. This is one of the most useful parts of cookstyle so this is a pretty major regression See chef/cookstyle#733 Treetop was monkeypatching String.blank? which was breaking the usage of Ruby's implementation by Cookstyle/RuboCop. This was causing: rubocop/rubocop#8636 Signed-off-by: Tim Smith <tsmith@chef.io>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
Chef Workstation 20.8.125 -
cookstyle
doesn't detect some Ruby stuffChef Workstation Version
Chef Workstation 20.8.125
was working with Chef Workstation 20.6.62
Platform Version
MacOS 10.15.6
Test Code
Current output
Chef Workstation 20.8.125 - Cookstyle 6.15.5, RuboCop 0.89.1
Expected output
Chef Workstation 20.6.62 - Cookstyle 6.7.3, RuboCop 0.85.0
The text was updated successfully, but these errors were encountered: