summaryrefslogtreecommitdiff
path: root/lib/chef/shell
Commit message (Collapse)AuthorAgeFilesLines
* Style/StringLiteralsInInterpolationLamont Granquist2019-07-051-1/+1
| | | | | | since we use double quotes, be consistent everywhere. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* Style/ClassCheckLamont Granquist2019-07-052-3/+3
| | | | | | convert kind_of? to is_a? Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* fix Layout/EmptyLineAfterGuardClauseLamont Granquist2019-07-021-0/+1
| | | | | | | i like this one, gives visual priority to returns or raises that are buried in the middle of things. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* more distro constantsMarc Chamberland2019-06-032-7/+8
| | | | Signed-off-by: Marc Chamberland <mchamberland@pbsc.com>
* more distro constants for "Chef"Marc Chamberland2019-05-231-3/+4
| | | | Signed-off-by: Marc Chamberland <mchamberland@pbsc.com>
* This hits the ones that are most frequently required.lcg/require-idempotencyLamont Granquist2019-05-201-2/+2
| | | | | | | | | Stops the kernel from scanning through all the activated gems every time in order to resolve names into pathnames. See rubygems/rubygems#2762 Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* Convert require to require_relativeLamont Granquist2019-05-083-19/+19
| | | | | | | This gives a speed boost since rubygems does not have to scan through every gem in the gemset in order to find the file. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* clean up whitespace againLamont Granquist2018-07-021-38/+38
| | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* fix Layout/IndentHeredocLamont Granquist2018-07-021-154/+154
| | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* fix Style/HashSyntaxLamont Granquist2018-07-022-29/+29
| | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* mechanical conversion of most debug log statements to traceThom May2018-03-261-1/+1
| | | | Signed-off-by: Thom May <thom@chef.io>
* Merge pull request #6718 from chef/chef14_deprecationsThom May2018-01-221-1/+1
|\ | | | | Remove previously deprecated mixins and Chef::DSL::Recipe::FullDSL
| * Remove previously deprecated mixins and Chef::DSL::Recipe::FullDSLchef14_deprecationsTim Smith2018-01-011-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | We wired these up previously with deprecations. They've been deprecated for a long time and are pretty rarely used on the Supermarket. Additionally we have Foodcritic rules to detect their usage: FC102: Deprecated Chef::DSL::Recipe::FullDSL class used FC100: Deprecated Chef::Mixin::Language mixin used FC099: Deprecated Chef::Mixin::LanguageIncludeRecipe mixin used FC098: Deprecated Chef::Mixin::RecipeDefinitionDSLCore mixin used FC097: Deprecated Chef::Mixin::LanguageIncludeAttribute mixin used Signed-off-by: Tim Smith <tsmith@chef.io>
* | prepping for rubocop 0.52.1lcg/rubocop-0.52.1Lamont Granquist2018-01-171-2/+3
|/ | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* Pass json configuration to ShellSession classJon Cowie2017-08-101-4/+4
| | | | | | This commit passes json configuration to ShellSession class and thusly into ChefClient object created by the ShellSession. This fixes an issue where runlists passed in via a json file using the -j flag would not be loaded when running chef-shell in client mode using the -z flag. Signed-off-by: Jon Cowie <jcowie@chef.io>
* Make chef-shell to use SoloSession properlySalim Afiune2017-03-171-0/+6
| | | | Signed-off-by: Salim Afiune <afiune@chef.io>
* Deprecate SoloSession into SoloLegacySessionSalim Afiune2017-03-171-3/+7
| | | | Signed-off-by: Salim Afiune <afiune@chef.io>
* fix specs: RedundantReturn, RedundantSelf, RedundantBeginLamont Granquist2017-02-132-4/+4
| | | | | | department of redundancy department Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* Output https links in chef-shellhttps_shellTim Smith2016-11-141-2/+2
| | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* fix some Style/ExtraSpacing that crept into this patchLamont Granquist2016-08-161-2/+2
| | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* "fix" Lint/NestedMethodDefinition copsLamont Granquist2016-08-162-3/+3
| | | | | | these are all awful, but fixing them will be time consuming. by disabling them on the existing offenses we can enable the cop to prevent more of this creeping into the codebase.
* Ensure that solo specific code is run at the proper timeThom May2016-05-161-3/+3
| | | | | | ie, only when we're in OG mode and not in local mode. Signed-off-by: Thom May <thom@chef.io>
* add some @yield docsLamont Granquist2016-03-041-0/+1
| | | | added some FIXMEs because trying to sort out types for random undocumented code was making me completely aggro
* remove unused block argumentslcg/chefstyle-perfLamont Granquist2016-02-231-1/+1
|
* convert MultilineOperationIndentation style to "indented"Lamont Granquist2016-02-121-3/+3
| | | | this looks nicer.
* autofixing whitespace copsLamont Granquist2016-02-053-35/+34
| | | | | | | | | | | | | | | | | | | | | | 4174 Style/SpaceInsideHashLiteralBraces 1860 Style/SpaceAroundOperators 1336 Style/SpaceInsideBlockBraces 1292 Style/AlignHash 997 Style/SpaceAfterComma 860 Style/SpaceAroundEqualsInParameterDefault 310 Style/EmptyLines 294 Style/IndentationConsistency 267 Style/TrailingWhitespace 238 Style/ExtraSpacing 212 Style/SpaceBeforeBlockBraces 166 Style/MultilineOperationIndentation 144 Style/TrailingBlankLines 120 Style/EmptyLineBetweenDefs 101 Style/IndentationWidth 82 Style/SpaceAroundBlockParameters 40 Style/EmptyLinesAroundMethodBody 29 Style/EmptyLinesAroundAccessModifier 1 Style/RescueEnsureAlignment
* Update all auth email address from opscode.com and getchef.com to chef.io.Noah Kantrowitz2016-02-022-2/+2
| | | Generated via git ls-files | xargs perl -pi -e "s/(Author.*?<[^@]+@)(?:opscode\\.com|getchef\\.com)(>)/\\1chef.io\\2/gi"
* Replace all Opscode copyrights with Chef Software.Noah Kantrowitz2016-02-022-2/+2
| | | Created via git ls-files | xargs perl -pi -e "s/(Copyright.*?), Opscode(,)? Inc(\.)?/\\1, Chef Software Inc./gi"
* Copyright year update for 2016 and massive cleanup.Noah Kantrowitz2016-02-023-4/+4
| | | Generated via git ls-files | xargs perl -pi -e "s/[Cc]opyright (?:\([Cc]\) )?((?\!$(date +%Y))\\d{4})(-\\d{4})?([, ][ \d]+)*(,|(?= ))/Copyright \\1-$(date +%Y),/g"
* Use double quotes by defaultThom May2016-01-143-23/+23
| | | | | | | This is an entirely mechanically generated (chefstyle -a) change, to go along with chef/chefstyle#5 . We should pick something and use it consistently, and my opinion is that double quotes are the appropriate thing.
* really fix copslcg/really-fix-copsLamont Granquist2016-01-121-34/+34
|
* chefstyle: fix Lint/StringConversionInInterpolationLamont Granquist2016-01-121-1/+1
| | | | useless use of `"#{foo.to_s}"`
* Convert all uses of Chef::REST to Chef::ServerAPItm/no_more_restThom May2016-01-113-32/+5
| | | | | | | In the process, stop auto-expanding JSON in the HTTP client, and let individual classes control that themselves. Fixes #2737, Fixes #3518
* Update branding from opscode.com to chef.ioJulian C. Dunn2015-02-081-2/+2
|
* Refactoring ResourceCollection interface to be backwards compatiable, all ↵tyler-ball2014-10-162-4/+4
| | | | specs continue to pass, removing ResourceSet and ResourceList from consumer knowledge
* Finishing all code changes to split resource collection into 2 data containerstyler-ball2014-10-162-4/+4
|
* Make FileVendor configuration specific to the two implementationsdanielsdeleo2014-07-301-2/+2
| | | | | | | | | | | | | | | | FileVendor previously was configured by storing a closure/anonymous function as a class instance variable. This had the following downsides: * The API was too general, which caused a lot of code repetition * The block was lazily evaluated, which hid errors and made testing more difficult * The closures captured references to classes with references to large data structures, which complicates GC. Since we've only ever had the same two implementations of FileVendor, we can encapsulate configuration of the FileVendor factory by wrapping each configuration option in a method. As a side benefit, arguments to these methods will be eagerly evaluated, which makes it easier to detect errors.
* [CHEF-5314] Support override_runlist CLI option in shef/chef-shellRyan Cragun2014-05-201-3/+3
|
* [CHEF-4673] Change docs URL in chef-shell.Julian C. Dunn2013-11-061-1/+1
|
* Normalize whitespace on all filesSeth Vargo2013-09-261-1/+1
|
* [CHEF-2467] Ensure Shef loads run_context using correct run_list and ↵Steven Danna2013-04-181-2/+2
| | | | | | | | environment. Currently, Shef in client-mode will not properly process attributes set in roles or environments since we assume an empty runlist and the _default environment when building the node.
* [OC-3564] move core Chef to the repo root \o/ \m/Seth Chisamore2012-10-304-0/+1039
The opscode/chef repository now only contains the core Chef library code used by chef-client, knife and chef-solo!