Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | syntax fixlcg/keepalives2 | Lamont Granquist | 2016-03-08 | 1 | -1/+1 |
| | |||||
* | keepalive rebase | Lamont Granquist | 2016-03-08 | 8 | -63/+206 |
| | |||||
* | Merge pull request #4673 from dduponchel/GH-2561 | Thom May | 2016-03-08 | 2 | -4/+33 |
|\ | | | | | preferred_manifest_record: fix pretty print. | ||||
| * | preferred_manifest_record: fix pretty print. | David Duponchel | 2016-03-07 | 2 | -4/+33 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The `preferred_manifest_record` function pretty prints the list of existing files when the asked file can't be found. To do that, the root path of the cookbook is removed from each existing file but the update is done on the `existing_files` variable, not on a copy. When called several times, this variable will eventually be equals to `[nil]`, leading to a NoMethodError. Without the fix, the new unit test fails with > expected Chef::Exceptions::FileNotFound, got #<NoMethodError: undefined method `[]' for nil:NilClass> with backtrace: > # ./lib/chef/cookbook_version.rb:321:in `block in preferred_manifest_record' > # ./lib/chef/cookbook_version.rb:319:in `map!' > # ./lib/chef/cookbook_version.rb:319:in `preferred_manifest_record' > # ./spec/unit/cookbook_version_file_specificity_spec.rb:318:in `block (3 levels) in <top (required)>' > # ./spec/unit/cookbook_version_file_specificity_spec.rb:317:in `block (2 levels) in <top (required)>' Fixes #2561. | ||||
* | | Bump version to 12.8.112.8.1jk/bump-version | John Keiser | 2016-03-07 | 4 | -6/+11 |
|/ | |||||
* | Merge pull request #4667 from chef/accept_docker | Matt Wrock | 2016-03-04 | 3 | -0/+16 |
|\ | | | | | adding docker to top_cookbooks | ||||
| * | adding docker to top_cookbooks | Matt Wrock | 2016-03-04 | 3 | -0/+16 |
| | | |||||
* | | Revert "Unpin github_changelog_generator" | Lamont Granquist | 2016-03-04 | 1 | -1/+1 |
| | | | | | | | | This reverts commit 57ea7d9278201d23c94397650a454281fe16fb7d. | ||||
* | | Unpin github_changelog_generator | John Keiser | 2016-03-04 | 1 | -1/+1 |
|/ | |||||
* | Merge pull request #4617 from chef/lcg/chefstyle-perf | Lamont Granquist | 2016-03-04 | 73 | -106/+156 |
|\ | | | | | Autofixing new Perf cops in 0.37.2 | ||||
| * | add some @yield docs | Lamont Granquist | 2016-03-04 | 16 | -5/+53 |
| | | | | | | | | added some FIXMEs because trying to sort out types for random undocumented code was making me completely aggro | ||||
| * | remove unused block argumentslcg/chefstyle-perf | Lamont Granquist | 2016-02-23 | 25 | -29/+29 |
| | | |||||
| * | Autofixing new Perf cops in 0.37.2 | Lamont Granquist | 2016-02-23 | 63 | -75/+77 |
| | | | | | | | | | | | | | | | | | | | | 6 Performance/Casecmp 18 Performance/Detect 1 Performance/RangeInclude 27 Performance/RedundantBlockCall 6 Performance/RedundantMatch 5 Performance/RedundantMerge 18 Performance/StringReplacement | ||||
* | | Merge pull request #4651 from chef/tball/rehash_error | Tyler Ball | 2016-03-04 | 8 | -4/+105 |
|\ \ | | | | | | | Always rehash from gem source and not existing hash file | ||||
| * | | Adding specs for rehash update | tyler-ball | 2016-03-04 | 6 | -3/+72 |
| | | | |||||
| * | | Changing the knife behavior when using the cached plugins to only display 1 ↵ | tyler-ball | 2016-03-04 | 1 | -3/+21 |
| | | | | | | | | | | | | error message when plugin files are missing from the cache | ||||
| * | | First part of fixing the rehash command, it now always loads gems from disc ↵ | tyler-ball | 2016-03-04 | 3 | -1/+15 |
|/ / | | | | | | | instead of using the hash file | ||||
* | | Merge pull request #4663 from chef/lcg/better-chef-rspec-constraints | Lamont Granquist | 2016-03-04 | 14 | -44/+39 |
|\ \ | | | | | | | better chef+ruby rspec constraints | ||||
| * | | remove merge conflict resolution errorlcg/better-chef-rspec-constraints | Lamont Granquist | 2016-03-03 | 1 | -4/+0 |
| | | | |||||
| * | | better chef+ruby rspec constraints | Lamont Granquist | 2016-03-03 | 14 | -42/+41 |
| | | | | | | | | | | | | | | | | | | | | | | | | lets us use Gem::Dependency constraints directly as rspec constraints for matching Chef::Version and RUBY_VERSION. copies some magic sauce from bundler that does something similar | ||||
* | | | Merge pull request #4664 from chef/salam/databag-fix | Thom May | 2016-03-04 | 2 | -3/+55 |
|\ \ \ | | | | | | | | | Data bag item hash can have name key | ||||
| * | | | Data bag item hash can have name keysalam/databag-fix | Salim Alam | 2016-03-03 | 2 | -3/+55 |
| | | | | |||||
* | | | | Merge pull request #4478 from chef/lcg/rfc-060-gem-metadata | Lamont Granquist | 2016-03-03 | 15 | -8/+245 |
|\ \ \ \ | |_|/ / |/| | | | RFC-060 gem metadata MVP | ||||
| * | | | need to require bundlerlcg/rfc-060-gem-metadata | Lamont Granquist | 2016-02-23 | 1 | -0/+1 |
| | | | | | | | | | | | | | | | | for Bundler::UI | ||||
| * | | | pull latest nightly instead of latest release | Lamont Granquist | 2016-02-17 | 1 | -0/+2 |
| | | | | | | | | | | | | | | | | | | | | | | | | if we're dependent upon fixing something in the omnibus build (i.e. the shipped version of appbundler) it makes sense to install over the latest nightly rather than the latest release. | ||||
| * | | | remove now-unnecessary require | Lamont Granquist | 2016-02-09 | 1 | -1/+0 |
| | | | | |||||
| * | | | add some metadata specs around 'gem' method | Lamont Granquist | 2016-02-09 | 1 | -0/+27 |
| | | | | |||||
| * | | | try to avoid the halite tests for now | Lamont Granquist | 2016-02-09 | 1 | -0/+4 |
| | | | | |||||
| * | | | rubocopisms | Lamont Granquist | 2016-02-09 | 1 | -2/+2 |
| | | | | |||||
| * | | | can't strip the PATH out of the env. | Lamont Granquist | 2016-02-09 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | | | | | also add back a bunch of other ruby env vars so that our ruby environment remains sane. | ||||
| * | | | fix broken specs | Lamont Granquist | 2016-02-09 | 2 | -1/+4 |
| | | | | |||||
| * | | | add comments and cleanup | Lamont Granquist | 2016-02-09 | 3 | -7/+14 |
| | | | | |||||
| * | | | remove unnecessary paranoia | Lamont Granquist | 2016-02-09 | 1 | -5/+1 |
| | | | | |||||
| * | | | make rubygems.org configurable | Lamont Granquist | 2016-02-09 | 3 | -2/+4 |
| | | | | | | | | | | | | | | | | for bonus points gem_package will pick this up as well. | ||||
| * | | | get the formatting for cookbook gems correct | Lamont Granquist | 2016-02-09 | 6 | -18/+119 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | this leverages the PR: https://github.com/bundler/bundler/pull/4245 to redirect output from bundler into chef's logger/formatter to get the UI right. if you don't have that in the external bundler you'll just get output on STDOUT. fix is to upgrade bundler. | ||||
| * | | | also update rubygems correctly | Lamont Granquist | 2016-02-09 | 1 | -0/+2 |
| | | | | |||||
| * | | | remove bundle_install override on halite | Lamont Granquist | 2016-02-09 | 1 | -1/+0 |
| | | | | |||||
| * | | | always update bundler | Lamont Granquist | 2016-02-09 | 1 | -0/+1 |
| | | | | |||||
| * | | | fixed my .vimrc auto-copyright header | Lamont Granquist | 2016-02-09 | 1 | -1/+1 |
| | | | | |||||
| * | | | spellng | Lamont Granquist | 2016-02-09 | 1 | -1/+1 |
| | | | | |||||
| * | | | set the floor to when bundler/inline was released | Lamont Granquist | 2016-02-09 | 1 | -2/+3 |
| | | | | |||||
| * | | | bundler is now a runtime dep | Lamont Granquist | 2016-02-09 | 1 | -0/+3 |
| | | | | |||||
| * | | | RFC-060 gem metadata MVP | Lamont Granquist | 2016-02-09 | 5 | -1/+90 |
| | | | | |||||
* | | | | Merge pull request #4654 from chef/salam/chef-cleanup | Salim Alam | 2016-03-03 | 1 | -0/+2 |
|\ \ \ \ | |_|/ / |/| | | | Cleanup static libs at build time | ||||
| * | | | Cleanup static libs at build timesalam/chef-cleanup | Salim Alam | 2016-03-03 | 1 | -0/+2 |
|/ / / | |||||
* | | | Merge pull request #4652 from chef/missing_chefdk_maintainers | Tyler Ball | 2016-03-03 | 2 | -3/+3 |
|\ \ \ | | | | | | | | | ChefDK was mistakenly left out of the Chef maintenance policy | ||||
| * | | | ChefDK was mistakenly left out of the Chef maintenance policymissing_chefdk_maintainers | tyler-ball | 2016-03-02 | 2 | -3/+3 |
| | | | | |||||
* | | | | Merge pull request #4655 from coderanger/solobag | Noah Kantrowitz | 2016-03-03 | 2 | -0/+6 |
|\ \ \ \ | | | | | | | | | | | Clearer exception for loading non-existent data bag items in solo mode. | ||||
| * | | | | Clearer exception for loading non-existent data bag items in solo mode. | Noah Kantrowitz | 2016-03-02 | 2 | -0/+6 |
| | | | | | |||||
* | | | | | Merge pull request #4657 from chef/tm/option_changelog_generator | Thom May | 2016-03-03 | 1 | -6/+10 |
|\ \ \ \ \ | | | | | | | | | | | | | guard against changelog generator not being installed |