| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
| |
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
|
|\
| |
| | |
pull rubocop 0.37.2 into chefstyle
|
| | |
|
| |
| |
| |
| | |
this looks nicer.
|
| |
| |
| |
| |
| | |
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 ensure source is used on all functional tests
|
| |
| |
| |
| | |
ensure source is used on all functional tests
|
|\ \
| |/
|/| |
remove Chef::Mixin::Command use
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
we don't use run_command anywhere and this can apparently cause uninitialized
constant issues when knife ssh is invoked via other knife commands (i.e.
bootstrap) where apparently the transitive lazy deps aren't getting
invoked correctly. there's another bug there, but we do not have any
run_command statements in knife anymore so we should be able to drop
this as a much easier fix.
closes #4546
|
|\ \
| |/
|/| |
fix a batch of cops
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Style/NegatedWhile
Style/ParenthesesAroundCondition
Style/WhileUntilDo
Style/WordArray
Performance/ReverseEach
Style/ColonMethodCall
|
| |
| |
| |
| | |
successful release pipeline run
|
|\ \
| | |
| | | |
removing rm -rf in chef-solo recipe_url
|
| | | |
|
| | | |
|
| | |
| | |
| | |
| | | |
this really closes #1515 and #1751
|
| |/
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
- 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
|
| | |
|
| | |
|
| |
| |
| |
| | |
fixes #4549
|
|\ \
| |/
|/| |
fix for https://github.com/chef/chef/issues/2113
|
| | |
|
| |\ |
|
| | | |
|
| | |
| | |
| | |
| | | |
second round of autofixing for some reason...
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
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.
|
|\ \ \
| | | |
| | | | |
auto-fixing some easy cops
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
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.
|
|\ \ \ \
| |/ / /
|/| | | |
|
| | | | |
|
| | | | |
|
| | | |
| | | |
| | | |
| | | | |
verbose log info.
|
| | | | |
|
|\ \ \ \
| | | | |
| | | | | |
fixes #4515 and use FormatMessage to get correct error messages from native API instead of retyping them with our fingers
|
| |/ / /
| | | |
| | | |
| | | | |
API instead of retyping them with our fingers
|
|\ \ \ \
| |/ / /
|/| | | |
Allow use of command line fips switch for knife
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This somewhat worked before. However, it was just for knife bootstrap.
It also didn't support --no-fips in the case where it was in your
knife.rb and you decided that you didn't want to use fips for
that one call.
The assumption here is fips mode you run knife with is the fips
mode the node will get. This has the nice property that validatorless
bootstraps will talk to the chef server in fips mode if the node
is requested to be in fips mode.
|
|\ \ \ \
| | | | |
| | | | | |
Copyright and author line update
|
| | | | | |
|
| | | | | |
|
| | | | |
| | | | |
| | | | | |
Generated via git ls-files | xargs perl -pi -e "s/(Author.*?<[^@]+@)(?:opscode\\.com|getchef\\.com)(>)/\\1chef.io\\2/gi"
|
| | | | |
| | | | |
| | | | | |
Created via git ls-files | xargs perl -pi -e "s/(Copyright.*?), Opscode(,)? Inc(\.)?/\\1, Chef Software Inc./gi"
|
| | | | | |
|
| | | | |
| | | | |
| | | | | |
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"
|
|/ / / /
| | | |
| | | |
| | | | |
profiles installed
|
|\ \ \ \
| | | | |
| | | | | |
correct windows 10 major/minor version numbers
|
| | | | | |
|