summaryrefslogtreecommitdiff
path: root/lib/chef/mixin
Commit message (Collapse)AuthorAgeFilesLines
* More Opscode -> Chef updatesTim Smith2016-10-201-1/+1
| | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* Fixed double quotes issue in powershell_outNimishaS2016-10-071-1/+1
| | | | Signed-off-by: NimishaS <nimisha.sharad@msystechnologies.com>
* Revert Handling double quotes in powershell_outMatt Wrock2016-10-051-10/+2
| | | | Signed-off-by: Matt Wrock <matt@mattwrock.com>
* Raising Mixlib::ShellOut::ShellCommandFailed exceptionNimishaS2016-10-031-2/+2
| | | | Signed-off-by: NimishaS <nimisha.sharad@msystechnologies.com>
* Handling double quotes. Added specsNimishaS2016-09-301-2/+10
| | | | Signed-off-by: NimishaS <nimisha.sharad@msystechnologies.com>
* Fixed undefined short_cksum method issue and checksum in uppercase issue for ↵Aliasgar162016-09-201-0/+5
| | | | | | | | | | windows_package resource. (#5332) * Fixed undefined short_cksum method issue and checksum in uppercase issue for windows_package resource. * Added RSpecs for the changes done here. * Fixed RSpecs issues.
* Allow flagging a resource property as sensitiveadamleff/sensitive-propertiesAdam Leff2016-08-251-0/+3
| | | | | | | | | | | | Some properties in custom resources may include sensitive data, such as a password for a database server. When the Resource's state is built for use by Data Collector or similar auditing tool, `Chef::Resource#state_for_resource_reporter` builds a hash of all state properties for that resource and their values. This leads to sensitive data being transmitted and potentially stored in the clear. This change enhances properties with the ability to set an individual property as sensitive and then have the value of that property suppressed when exporting the Resource's state.
* clean up everythingLamont Granquist2016-08-231-0/+30
| | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* Stop using obsolete URI.escape; use Addressable::URI instead.John Keiser2016-08-221-1/+2
| | | | See http://stackoverflow.com/questions/2824126/whats-the-difference-between-uri-escape-and-cgi-escape#answer-13059657 for details on why.
* fix Style/BlockDelimiters, Style/MultilineBlockLayout and 0.42.0 engine upgradeLamont Granquist2016-08-173-5/+5
| | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* fixing Style/MultilineTernaryOperator copLamont Granquist2016-08-161-4/+3
| | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* make deprecation warnings actual deprecation warningsLamont Granquist2016-04-251-1/+1
|
* remove duping the args hashlcg/shell_out_syntaxLamont Granquist2016-04-251-1/+0
| | | | | we only ever did this in order to mutate the options and with the **options syntax we don't need to do this anymore.
* tighter code from @coderangerLamont Granquist2016-04-251-10/+5
|
* modernize shell_out method syntaxLamont Granquist2016-04-251-16/+11
| | | | **options has worked ever since we've deprecated ruby 1.9.x
* remove global_resource_collectionLamont Granquist2016-04-111-4/+0
|
* readability tweakLamont Granquist2016-04-111-2/+4
|
* fix logic in updating new_resourceLamont Granquist2016-04-111-3/+4
|
* add notifying_block and subcontext_block to chefLamont Granquist2016-04-111-0/+54
|
* Obvious fix: Update lamont@getchef.com email to lamont@chef.ioadamedx/lamont-emailAdam Edwards2016-04-041-1/+1
|
* Fix proxified socketSalim Alam2016-03-211-2/+6
|
* lazy module inclusion into DSL moduleslcg/use-and-lazy-module-includeLamont Granquist2016-03-151-0/+77
| | | | | | Chef::DSL::Recipe::FullDSL.send(:include, MyModule) will now patch all its descendants that it has been included into (works the way actual inheritance works now).
* Merge pull request #4706 from chef/lcg/chefstyle-andorLamont Granquist2016-03-153-3/+3
|\ | | | | chefstyle Style/AndOr fixes
| * chefstyle Style/AndOr fixesLamont Granquist2016-03-143-3/+3
| | | | | | | | | | this is part of our informal style guide, lets make it formal since clearly its not getting followed very well.
* | Issue 4334: Because set(property, nil) gets turned into get(property), add a ↵cd/fix-spurious-warningsChris Doherty2016-03-111-2/+2
|/ | | | 'nil_set' parameter to get() to skip a spurious warning when that happens.
* Fix property coercion in both places, because they were different and Noah Kantrowitz2016-03-091-1/+1
| | | one in Chef::Mixin::Properties would result in duplicated values.
* Merge pull request #4617 from chef/lcg/chefstyle-perfLamont Granquist2016-03-044-5/+8
|\ | | | | Autofixing new Perf cops in 0.37.2
| * add some @yield docsLamont Granquist2016-03-042-0/+2
| | | | | | | | 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-232-2/+2
| |
| * Autofixing new Perf cops in 0.37.2Lamont Granquist2016-02-233-3/+4
| | | | | | | | | | | | | | | | | | | | 6 Performance/Casecmp 18 Performance/Detect 1 Performance/RangeInclude 27 Performance/RedundantBlockCall 6 Performance/RedundantMatch 5 Performance/RedundantMerge 18 Performance/StringReplacement
* | ensure paths maintain utf-8ness in non ascii encodingsumlautMatt Wrock2016-02-241-1/+1
|/
* flip multiline function arguments aroundlcg/rubocop-0.37.2Lamont Granquist2016-02-193-7/+7
| | | | no enforced trailing comma on arguments...
* deal with 0.37.2 renamed copsLamont Granquist2016-02-182-2/+2
| | | | | | | | | | | | | | | 252 Style/TrailingCommaInLiteral 84 Style/TrailingCommaInArguments 15 Style/SpaceAroundKeyword -- 351 Total We already dealt with SpaceAroundKeyword under its old name SpaceBeforeModifierKeyword, it looks like it got stricter about spaces after keywords. TrailingComma also got split, and it looks like the TrailingCommaInArguments behavior is new?
* convert MultilineOperationIndentation style to "indented"Lamont Granquist2016-02-122-2/+2
| | | | this looks nicer.
* pull rubocop 0.37.2 into chefstyleLamont Granquist2016-02-122-6/+6
| | | | | this is from the same ruleset that we had, but the new code catches more conditions.
* autofixing auto-inserted delimiterslcg/chefstyle-batchLamont Granquist2016-02-092-4/+4
|
* auto fixing some rubocopsLamont Granquist2016-02-093-6/+6
| | | | | | | | | Style/NegatedWhile Style/ParenthesesAroundCondition Style/WhileUntilDo Style/WordArray Performance/ReverseEach Style/ColonMethodCall
* autofixing whitespace copsLamont Granquist2016-02-0521-96/+81
| | | | | | | | | | | | | | | | | | | | | | 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
* Merge branch 'smurawski/dsc_resource_converge_logging'Steven Murawski2016-02-031-0/+2
|\
| * Allow PowerShellTypeCoercions to translate Chef::Node::ImmutableArraysmurawski/dsc_resource_converge_loggingSteven Murawski2016-02-031-1/+2
| |
| * Allow PowerShellTypeCoercions to translate Chef::Node::ImmutableMashSteven Murawski2016-02-031-0/+1
| |
* | Allow multiple property_types in `is`jk/property-is-multiple-typesJohn Keiser2016-02-031-9/+20
|/
* Update for chef.io domain in copyright lines too.Noah Kantrowitz2016-02-021-1/+1
|
* Update all auth email address from opscode.com and getchef.com to chef.io.Noah Kantrowitz2016-02-0228-34/+34
| | | 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-0224-24/+24
| | | 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-0236-41/+41
| | | 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"
* Fix nil with properties:John Keiser2016-01-271-1/+11
| | | | | | 1. Warn when default values are invalid. 2. Never validate nil (on set or get) if there is no default. 3. Emit "will be invalid in Chef 13" warning when setting an invalid nil value.
* Autofixing Style/PercentLiteralDelimeterslcg/percentliteraldelimetersLamont Granquist2016-01-181-2/+2
| | | | | | See chef/chefstyle#11 for analysis and discussion. We select '{}' since audit of our source code shows that is the most common, and that used to be the dominant learning paradigm (e.g. in ruby 1.9 pickaxe book.
* Use double quotes by defaultThom May2016-01-1428-87/+87
| | | | | | | 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.
* autocorrecting Style/TrailingCommalcg/trailing_commaLamont Granquist2016-01-133-11/+11
| | | | chefstyle -a fixed 1044 occurrances
View Lorry Controller Status