delta/ruby-gems/chef.git/lib/chef/knife/core, branch https github.com: opscode/chef.git Replace highline gem with tty-prompt 2020-07-21T11:17:30+00:00 Vivek Singh vivek.singh@msystechnologies.com 2020-07-08T11:07:29+00:00 494ca6525b72a594df39f4c39861b4feb63d85fe Signed-off-by: Vivek Singh <vivek.singh@msystechnologies.com> 
Signed-off-by: Vivek Singh <vivek.singh@msystechnologies.com>
Move the rehash methods into the subcommand loader 2020-07-15T01:01:01+00:00 Tim Smith tsmith84@gmail.com 2020-07-15T01:01:01+00:00 297c549dc21c90db806478a526f2167c8769a600 This way we can use them in chef-cli to rehash any time we install a gem. Signed-off-by: Tim Smith <tsmith@chef.io> 
This way we can use them in chef-cli to rehash any time we install a
gem.

Signed-off-by: Tim Smith <tsmith@chef.io>
Fix quotes for Chefstyle 2020-07-08T00:32:22+00:00 Tim Smith tsmith84@gmail.com 2020-07-07T22:54:52+00:00 6c4c32340ac0fcebb53768cb78f5760e55c35514 Signed-off-by: Tim Smith <tsmith@chef.io> 
Signed-off-by: Tim Smith <tsmith@chef.io>
expand_path with __dir__ instead of __FILE__ 2020-07-08T00:30:49+00:00 Tim Smith tsmith84@gmail.com 2020-07-07T22:31:02+00:00 ed460911a49716817ded2986e81a5116dfbb795e It's easier to read this at a glance and understand what path we're trying to get. Signed-off-by: Tim Smith <tsmith@chef.io> 
It's easier to read this at a glance and understand what path we're trying to get.

Signed-off-by: Tim Smith <tsmith@chef.io>
Merge pull request #10123 from chef/assignment 2020-07-07T22:53:58+00:00 Tim Smith tsmith@chef.io 2020-07-07T22:53:58+00:00 e42b214c6c371558dd6de1eb02f89c91f9ce9340 Avoid assigning variables before returning if we don't have to 
Avoid assigning variables before returning if we don't have to
 Add spaces after attrs 2020-07-07T22:28:50+00:00 Tim Smith tsmith84@gmail.com 2020-07-07T22:28:50+00:00 d7c4085d55a537d297f557c119fa6a9bd7e22690 This makes it easier to read the classes. Signed-off-by: Tim Smith <tsmith@chef.io> 
This makes it easier to read the classes.

Signed-off-by: Tim Smith <tsmith@chef.io>
Avoid assigning variables before returning if we don't have to 2020-07-07T22:19:21+00:00 Tim Smith tsmith84@gmail.com 2020-07-07T22:19:21+00:00 1485a862053fb20ade1f93bb82c6918cd4078975 These seemed pretty straight forward. There's a new Rubocop cop for these. Signed-off-by: Tim Smith <tsmith@chef.io> 
These seemed pretty straight forward. There's a new Rubocop cop for these.

Signed-off-by: Tim Smith <tsmith@chef.io>
Fix more typos 2020-06-23T21:48:25+00:00 Tim Smith tsmith84@gmail.com 2020-06-19T01:00:51+00:00 d71346347e518137028641f5a4448a31450ebc79 Signed-off-by: Tim Smith <tsmith@chef.io> 
Signed-off-by: Tim Smith <tsmith@chef.io>
Use .match? not =~ when match values aren't necessary 2020-06-11T22:32:53+00:00 Tim Smith tsmith84@gmail.com 2020-06-11T22:02:42+00:00 5825eea4b139b9af089c3075d042d5d3160583ec Autocorrected from RuboCop Performance which is now smart enough to detect when you use the match and when you don't. Using match? does not create any objects so it's slightly faster and uses less memory. Signed-off-by: Tim Smith <tsmith@chef.io> 
Autocorrected from RuboCop Performance which is now smart enough to detect when you use the match and when you don't. Using match? does not create any objects so it's slightly faster and uses less memory.

Signed-off-by: Tim Smith <tsmith@chef.io>
Force the bootstrap_directory to use the right separator on Windows 2020-05-20T17:41:25+00:00 Bryan McLellan btm@loftninjas.org 2020-05-20T17:17:42+00:00 5c3fcc525332a45e80fc3cc5d550ae7866eb9364 ChefConfig::Config.etc_chef_dir and c_chef_dir would normally replace the standard forward slash separator with the backslash separator for Windows using PathHelper.cleanpath. However that has conditionals checking the platform and there's no clean way to modify those conditionals to take into account connection.platform in knife bootstrap when appropriate. This is a simple solution without mucking up the method definition of any more methods in chef-config. Signed-off-by: Bryan McLellan <btm@loftninjas.org> 
ChefConfig::Config.etc_chef_dir and c_chef_dir would normally replace
the standard forward slash separator with the backslash separator for
Windows using PathHelper.cleanpath. However that has conditionals
checking the platform and there's no clean way to modify those
conditionals to take into account connection.platform in knife bootstrap
when appropriate.

This is a simple solution without mucking up the method definition of
any more methods in chef-config.

Signed-off-by: Bryan McLellan <btm@loftninjas.org>