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
Don't format Floats as Strings #54
Conversation
Previously if the only change in a class was a parameter's data type from a `Float` to a `String`, then the class resource would be marked as different, but [str_diff](https://github.com/camptocamp/puppet-catalog-diff/blob/b7f3031e380502eb6154242b1710b9ab7c105b42/lib/puppet/catalog-diff/comparer.rb#L130) would return `nil` causing a `Error: undefined method split' for nil:NilClass` error [here](https://github.com/camptocamp/puppet-catalog-diff/blob/b7f3031e380502eb6154242b1710b9ab7c105b42/lib/puppet/catalog-diff/comparer.rb#L60) Fixes voxpupuli#32
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
Dear stale bot. This is actually a fairly trivial PR that fixes an actual issue. Labeling it |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
This is a legit fix. Hopefully someone will merge it soon so stale bot won't strike again! |
@bastelfreak Have you got any update on any migration plans? |
I am waiting for feedback in #60 |
@bastelfreak Would you like to merge this one now? |
@alexjfisher Migration Done. |
Yep. Just noticed. Didn't want to merge my own PR though. |
Previously if the only change in a class was a parameter's data type
from a
Float
to aString
, then the class resource would be marked asdifferent, but str_diff would return
nil
causing aError: undefined method split' for nil:NilClass
error hereFixes #32