diff options
| author | Lamont Granquist <lamont@scriptkiddie.org> | 2016-05-02 16:08:54 -0700 |
|---|---|---|
| committer | Lamont Granquist <lamont@scriptkiddie.org> | 2016-05-02 16:08:54 -0700 |
| commit | b447bbf40017d3f8d1c172fda6c49c3e860edf24 (patch) | |
| tree | a626b0c8606874c11e432f616d5438a05a35fd7d | |
| parent | 51b507e1fea6514438344dd86edd4b95ead662d4 (diff) | |
| download | chef-b447bbf40017d3f8d1c172fda6c49c3e860edf24.tar.gz | |
tweak comment
| -rw-r--r-- | lib/chef/provider/package.rb | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/lib/chef/provider/package.rb b/lib/chef/provider/package.rb index 7df74c2f60..6d67cdbbb2 100644 --- a/lib/chef/provider/package.rb +++ b/lib/chef/provider/package.rb @@ -267,12 +267,13 @@ class Chef # The current_version should probably be dropped out of the method signature, it should # always be the first argument. # - # The name is not just bad, but completely misleading, consider: + # The name is not just bad, but i find it completely misleading, consider: # # target_version_already_installed?(current_version, new_version) + # target_version_already_installed?(current_version, candidate_version) # - # does not involve any comparison using the target_version but the target_version is - # in the method name. + # which of those is the 'target_version'? i'd say the new_version and i'm confused when + # i see it called with the candidate_version. # # `current_version_equals?(version)` would be a better name def target_version_already_installed?(current_version, target_version) |