Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | enable HTTP winrm listener for dsc functional testsdsc_listener | Matt Wrock | 2016-02-23 | 1 | -0/+10 |
| | |||||
* | fixed exe package downloads | Matt Wrock | 2016-02-22 | 1 | -1/+13 |
| | |||||
* | flip multiline function arguments aroundlcg/rubocop-0.37.2 | Lamont Granquist | 2016-02-19 | 58 | -256/+256 |
| | | | | no enforced trailing comma on arguments... | ||||
* | deal with 0.37.2 renamed cops | Lamont Granquist | 2016-02-18 | 71 | -295/+295 |
| | | | | | | | | | | | | | | | 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? | ||||
* | skip uid check for aix in user add functional tests | Matt Wrock | 2016-02-18 | 1 | -16/+9 |
| | |||||
* | Merge pull request #4111 from mikedodge04/launchd | Phil Dibowitz | 2016-02-17 | 2 | -0/+220 |
|\ | | | | | launchd for osx | ||||
| * | bundle exec chefstyle -a | Mike Dodge | 2016-02-15 | 2 | -51/+50 |
| | | |||||
| * | fixed provider spec | Mike Dodge | 2016-02-15 | 1 | -8/+11 |
| | | |||||
| * | added specs | Mike Dodge | 2016-02-15 | 2 | -0/+218 |
| | | |||||
* | | Merge pull request #4574 from chef/lcg/cookbook-shadow-deprecation | Lamont Granquist | 2016-02-16 | 2 | -2/+13 |
|\ \ | | | | | | | Extend cookbook shadowing deprecation warnings more broadly | ||||
| * | | don't warn about shadow cookbooks twice | Lamont Granquist | 2016-02-16 | 1 | -1/+3 |
| | | | | | | | | | | | | knife cookbook upload already warns once | ||||
| * | | fix specs | Lamont Granquist | 2016-02-12 | 1 | -1/+10 |
| | | | |||||
* | | | change shell to /bin/sh in user_add tests to fix aix | Matt Wrock | 2016-02-16 | 1 | -1/+1 |
| | | | |||||
* | | | Merge pull request #4586 from chef/tm/trailing_whitespace | Thom May | 2016-02-16 | 1 | -1/+1 |
|\ \ \ | | | | | | | | | fix trailing whitespace | ||||
| * | | | fix trailing whitespacetm/trailing_whitespace | Thom May | 2016-02-16 | 1 | -1/+1 |
| | | | | |||||
* | | | | Merge pull request #4582 from chef/tm/prepend-chef | Thom May | 2016-02-16 | 1 | -0/+85 |
|\ \ \ \ | |/ / / |/| | | | Prepend chef to tempfile name | ||||
| * | | | add tempfile teststm/prepend-chef | Thom May | 2016-02-16 | 1 | -0/+85 |
| | |/ | |/| | |||||
* | | | Merge pull request #4427 from juliandunn/remove-duplicate-shellcmddef | Lamont Granquist | 2016-02-16 | 2 | -17/+20 |
|\ \ \ | | | | | | | | | Refactor test to remove duplicate definition of ShellCmdResult | ||||
| * | | | Refactor test to remove duplicate definition of ShellCmdResult | Julian C. Dunn | 2016-01-18 | 2 | -17/+20 |
| | | | | |||||
* | | | | Merge pull request #4465 from artursitarski/change-shell-for-locked-user | Thom May | 2016-02-16 | 2 | -4/+7 |
|\ \ \ \ | | | | | | | | | | | Change shell for locked user. | ||||
| * | | | | Updated spec | artursitarski | 2016-02-02 | 2 | -4/+7 |
| | | | | | |||||
* | | | | | Merge pull request #4524 from PatrickWalker/feature/4154 | Steven Murawski | 2016-02-16 | 1 | -1/+1 |
|\ \ \ \ \ | | | | | | | | | | | | | Update to Log Level when showing unencrypted databag | ||||
| * | | | | | Update to Log Level when showing unencrypted databag | Patrick Walker | 2016-02-03 | 1 | -1/+1 |
| | | | | | | |||||
* | | | | | | Merge pull request #4579 from chef/cmluciano/fixhttperror | Thom May | 2016-02-16 | 1 | -2/+14 |
|\ \ \ \ \ \ | | | | | | | | | | | | | | | Make sure we have a valid object before calling close! | ||||
| * | | | | | | Add test to ensure Tempfile does the right thingcmluciano/fixhttperror | Thom May | 2016-02-16 | 1 | -2/+14 |
| | |_|_|/ / | |/| | | | | |||||
* | | | | | | test that splitting a string on commas DTRTtm/split_group_members | Thom May | 2016-02-16 | 1 | -0/+5 |
|/ / / / / | |||||
* | | | | | Be way more explicit about how we're handling datatm/missed_migration | Thom May | 2016-02-15 | 15 | -21/+28 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This just codifies the behaviour we're actually using - that we're passing a json string and expecting a hash back. Also adds a deprecation warning to the use of Chef::JSONCompat.from_json | ||||
* | | | | | Merge pull request #4573 from chef/lcg/bump-rubocop | Lamont Granquist | 2016-02-12 | 12 | -185/+185 |
|\ \ \ \ \ | | | | | | | | | | | | | pull rubocop 0.37.2 into chefstyle | ||||
| * | | | | | convert MultilineOperationIndentation style to "indented" | Lamont Granquist | 2016-02-12 | 7 | -26/+26 |
| | | | | | | | | | | | | | | | | | | | | | | | | this looks nicer. | ||||
| * | | | | | pull rubocop 0.37.2 into chefstyle | Lamont Granquist | 2016-02-12 | 10 | -168/+168 |
| | |_|_|/ | |/| | | | | | | | | | | | | | | | | | | this is from the same ruleset that we had, but the new code catches more conditions. | ||||
* | | | | | Fix policyfile testsad/fix_policyfile_zero | andy-dufour | 2016-02-12 | 1 | -2/+6 |
|/ / / / | |||||
* | | | | do not include source parameter when removing a chocolatey package and ↵no_web | Matt Wrock | 2016-02-11 | 1 | -4/+10 |
| | | | | | | | | | | | | | | | | ensure source is used on all functional tests | ||||
* | | | | Merge pull request #4541 from chef/lcg/chefstyle-batch | Lamont Granquist | 2016-02-11 | 109 | -391/+391 |
|\ \ \ \ | | | | | | | | | | | fix a batch of cops | ||||
| * | | | | autofixing auto-inserted delimiterslcg/chefstyle-batch | Lamont Granquist | 2016-02-09 | 86 | -333/+333 |
| | | | | | |||||
| * | | | | auto fixing some rubocops | Lamont Granquist | 2016-02-09 | 109 | -391/+391 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Style/NegatedWhile Style/ParenthesesAroundCondition Style/WhileUntilDo Style/WordArray Performance/ReverseEach Style/ColonMethodCall | ||||
* | | | | | Respect TMPDIR in functional tests | John Keiser | 2016-02-10 | 5 | -13/+13 |
| | | | | | |||||
* | | | | | Merge pull request #4545 from chef/lcg/remove-rm-rf | Lamont Granquist | 2016-02-10 | 2 | -44/+29 |
|\ \ \ \ \ | | | | | | | | | | | | | removing rm -rf in chef-solo recipe_url | ||||
| * | | | | | remove rm -rf in chef solo recipe_url | Lamont Granquist | 2016-02-10 | 2 | -44/+29 |
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - deprecates '-r' used for the recipe_url in chef-solo - adds --delete-entire-chef-repo option for users who want the old behavior back. - cleans up some old code closes #3802 closes #1515 closes #1751 | ||||
* | | | | | Ensure we do streaming requests correctly everywheretm/site_download_regression | Thom May | 2016-02-09 | 1 | -5/+5 |
| | | | | | |||||
* | | | | | Use a streaming request to download cookbook | Thom May | 2016-02-09 | 1 | -6/+4 |
|/ / / / | | | | | | | | | | | | | fixes #4549 | ||||
* | | | | autofixing some missed cops | Lamont Granquist | 2016-02-05 | 2 | -3/+3 |
| | | | | | | | | | | | | | | | | second round of autofixing for some reason... | ||||
* | | | | autofixing whitespace cops | Lamont Granquist | 2016-02-05 | 433 | -3520/+3223 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 | ||||
* | | | | Refactor rubygems_spec test and make it fastjdm/rubygems-test | Kartik Null Cating-Subramanian | 2016-02-04 | 1 | -254/+312 |
| | | | | |||||
* | | | | WIP make rubygems unit specs unit like | Jay Mundrawala | 2016-02-04 | 1 | -10/+49 |
| | | | | |||||
* | | | | Chocolatey provider: Raise a MissingLibrary exception if we can't find ↵cd/choco-error-msg | Chris Doherty | 2016-02-03 | 1 | -1/+42 |
| | | | | | | | | | | | | | | | | Chocolatey. | ||||
* | | | | Merge pull request #4523 from chef/chefstyle/fixes | Lamont Granquist | 2016-02-03 | 6 | -6/+6 |
|\ \ \ \ | |/ / / |/| | | | auto-fixing some easy cops | ||||
| * | | | auto-fixing some easy copschefstyle/fixes | Lamont Granquist | 2016-02-03 | 6 | -6/+6 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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. | ||||
* | | | | Merge branch 'smurawski/dsc_resource_converge_logging' | Steven Murawski | 2016-02-03 | 2 | -9/+22 |
|\ \ \ \ | |/ / / |/| | | | |||||
| * | | | Allow PowerShellTypeCoercions to translate Chef::Node::ImmutableArraysmurawski/dsc_resource_converge_logging | Steven Murawski | 2016-02-03 | 1 | -0/+5 |
| | | | | |||||
| * | | | Allow PowerShellTypeCoercions to translate Chef::Node::ImmutableMash | Steven Murawski | 2016-02-03 | 1 | -9/+15 |
| | | | |