summaryrefslogtreecommitdiff
path: root/lib/chef/provider
Commit message (Collapse)AuthorAgeFilesLines
* flip multiline function arguments aroundlcg/rubocop-0.37.2Lamont Granquist2016-02-1912-30/+30
| | | | no enforced trailing comma on arguments...
* deal with 0.37.2 renamed copsLamont Granquist2016-02-1814-21/+21
| | | | | | | | | | | | | | | 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?
* zypper multipackage performance fixlcg/fix-zypperLamont Granquist2016-02-171-1/+1
| | | | | helps to remember the values of expensive things that we go off and compute.
* Merge pull request #4111 from mikedodge04/launchdPhil Dibowitz2016-02-171-0/+208
|\ | | | | launchd for osx
| * bundle exec chefstyle -aMike Dodge2016-02-151-52/+52
| |
| * made requested changesMike Dodge2016-02-151-50/+60
| |
| * fixed rebaseMike Dodge2016-02-151-1/+1
| |
| * added specsMike Dodge2016-02-151-11/+4
| |
| * launchdMike Dodge2016-02-151-0/+205
| |
* | Merge pull request #4465 from artursitarski/change-shell-for-locked-userThom May2016-02-161-2/+2
|\ \ | | | | | | Change shell for locked user.
| * | More consistent wayartursitarski2016-01-271-4/+2
| | |
| * | Change shell for locked user.artursitarski2016-01-271-0/+2
| | |
* | | Merge branch 'route_logic_fix' of https://github.com/petracvv/chef into ↵Thom May2016-02-161-1/+1
|\ \ \ | |_|/ |/| | | | | petracvv-route_logic_fix
| * | Don't use routes with action :delete when writing configMihai Petracovici2015-08-251-1/+1
| | |
* | | convert MultilineOperationIndentation style to "indented"Lamont Granquist2016-02-125-14/+14
| | | | | | | | | | | | this looks nicer.
* | | pull rubocop 0.37.2 into chefstyleLamont Granquist2016-02-123-11/+11
| | | | | | | | | | | | | | | this is from the same ruleset that we had, but the new code catches more conditions.
* | | do not include source parameter when removing a chocolatey package and ↵no_webMatt Wrock2016-02-111-3/+4
| | | | | | | | | | | | ensure source is used on all functional tests
* | | autofixing auto-inserted delimiterslcg/chefstyle-batchLamont Granquist2016-02-097-8/+8
| | |
* | | auto fixing some rubocopsLamont Granquist2016-02-0915-20/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | Style/NegatedWhile Style/ParenthesesAroundCondition Style/WhileUntilDo Style/WordArray Performance/ReverseEach Style/ColonMethodCall
* | | autofixing whitespace copsLamont Granquist2016-02-0575-446/+443
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | Chocolatey provider: Raise a MissingLibrary exception if we can't find ↵cd/choco-error-msgChris Doherty2016-02-031-8/+28
| | | | | | | | | | | | Chocolatey.
* | | Merge pull request #4523 from chef/chefstyle/fixesLamont Granquist2016-02-033-3/+3
|\ \ \ | | | | | | | | auto-fixing some easy cops
| * | | auto-fixing some easy copschefstyle/fixesLamont Granquist2016-02-033-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3 Performance/CaseWhenSplat: Place when conditions with a splat at the end of the when branches. 1 Style/ElseAlignment: Align else with if. 1 Style/EvenOdd: Replace with Fixnum#even?. 1 Style/MethodDefParentheses: Use def with parentheses when there are parameters. 3 Style/PercentQLiterals: Do not use %Q unless interpolation is needed. Use %q. 3 Style/SpaceAfterColon: Space missing after colon. 2 Style/SpaceAfterMethodName: Do not put a space between a method name and the opening parenthesis. 2 Style/Tab: Tab detected. 1 Style/TrailingUnderscoreVariable: Do not use trailing _s in parallel assignment. 1 Style/UnneededCapitalW: Do not use %W unless interpolation is needed. If not, use %w.
* | | | include verbose output from the set method and shared function for accessing ↵Steven Murawski2016-02-031-12/+18
|/ / / | | | | | | | | | verbose log info.
* | | Merge pull request #4518 from coderanger/copyright-2016Noah Kantrowitz2016-02-02120-189/+189
|\ \ \ | | | | | | | | Copyright and author line update
| * | | 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-0248-60/+60
| | | | | | | | | | | | 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-0258-58/+58
| | | | | | | | | | | | Created via git ls-files | xargs perl -pi -e "s/(Copyright.*?), Opscode(,)? Inc(\.)?/\\1, Chef Software Inc./gi"
| * | | Manual fix ups for things that didn't follow the usual pattern.Noah Kantrowitz2016-02-023-3/+3
| | | |
| * | | Copyright year update for 2016 and massive cleanup.Noah Kantrowitz2016-02-02117-126/+126
| | | | | | | | | | | | 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"
* | | | Bad test lead to a provider coverage failure for brand new machines with no ↵Nate Walck2016-02-021-4/+7
|/ / / | | | | | | | | | profiles installed
* | | Merge pull request #4422 from chef/tm/apt_updateThom May2016-01-271-0/+78
|\ \ \ | |_|/ |/| | Add an apt_update resource
| * | respond to review commentsThom May2016-01-221-1/+2
| | |
| * | Dependency inject file locationsThom May2016-01-211-16/+21
| | |
| * | ensure that periodic files are created firstThom May2016-01-201-8/+18
| | |
| * | Add periodic action as the defaultThom May2016-01-201-1/+23
| | |
| * | Add an apt_update resourceThom May2016-01-201-0/+40
| | |
* | | only require what we needlcg/zypper-multipackageLamont Granquist2016-01-251-3/+1
| | |
* | | zypper multipackage patchLamont Granquist2016-01-251-39/+52
|/ /
* | Autofixing Style/PercentLiteralDelimeterslcg/percentliteraldelimetersLamont Granquist2016-01-1820-22/+22
| | | | | | | | | | | | 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.
* | fix chefstyle failuresLamont Granquist2016-01-151-1/+1
| | | | | | | | | | | | | | | | autocorrects some merges that will be broken when the next chefstyle is released. also runs chefstyle after the travis tests, because i'd rather know how the code is broken rather than how its not pretty.
* | fix candidate version filteringMatt Wrock2016-01-141-11/+22
| |
* | Use double quotes by defaultThom May2016-01-14108-641/+641
| | | | | | | | | | | | | | 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-1315-32/+32
| | | | | | | | chefstyle -a fixed 1044 occurrances
* | Merge pull request #4390 from chef/tm/fix_failLamont Granquist2016-01-131-1/+1
|\ \ | | | | | | Correct uses of fail to raise
| * | Correct uses of fail to raisetm/fix_failThom May2016-01-131-1/+1
| | | | | | | | | | | | Style/SignalException - 3
* | | debug log messages upon reloaded resourcesif/warn-on-reload-resourceIsa Farnik2016-01-121-1/+1
|/ / | | | | | | tests for debug messages when reloading resources
* | Merge pull request #4384 from chef/lcg/really-fix-copsLamont Granquist2016-01-128-73/+73
|\ \ | | | | | | really fix cops
| * | really fix copslcg/really-fix-copsLamont Granquist2016-01-128-73/+73
| | |
* | | Merge pull request #4378 from chef/choco_editsMatt Wrock2016-01-121-2/+18
|\ \ \ | | | | | | | | assert candidates exist for alternate sources and when pinning versions
View Lorry Controller Status