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
[CVE-2017-8418] - updating rubocop dependency. #8
Conversation
8d2c3f0
to
c64ac48
Compare
bin/metrics-xen.rb
Outdated
|
||
def run | ||
xen = metrics_hash | ||
xm = xm_hash | ||
xencount = xenservers | ||
i = 0 | ||
xen.each do |_k, v| | ||
# TODO: come back and refactor me | ||
xen.each_value do |_k, v| # rubocop:disable Metrics/BlockLength |
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.
I think each_value
only passes one param (the value of each key) into the block.
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.
Good catch, was too quick to look at the rubocop correction closely enough. I will get that fixed up.
c64ac48
to
97151a9
Compare
@amdprophet should be good now, let me know if you spot anything else. |
Breaking Changes: - removed ruby `< 2.1` support Misc: - appeased the cops - fixed bug with single quotes where interpolation is needed - update changelog guidelines link Signed-off-by: Ben Abrams <me@benabrams.it>
97151a9
to
cc035f0
Compare
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.
LGTM
Breaking Changes:
< 2.1
supportMisc:
Signed-off-by: Ben Abrams me@benabrams.it
Pull Request Checklist
sensu-plugins/community#77
General
Update Changelog following the conventions laid out on Keep A Changelog
Update README with any necessary configuration snippets
Binstubs are created if needed
RuboCop passes
Existing tests pass
Purpose
Resolve CVE, see parent issue for details
Known Compatibility Issues
Removes ruby
< 2.1
support