summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Support array args on windowslcg/win-array-argsLamont Granquist2019-06-063-3/+127
| | | | | | | | | | | | | | | | | | | | | | Implements correct quoting and escaping of arguments on windows. That means that this works right now: ```ruby filename = "c:\program files" shell_out("dir", filename); ``` So all the defensive coding around quotes-around-filepaths -- which were all actually buggy even when they worked (trailing backslashes would fail) -- are unnecessary and arguments can just be passed in as an array and this code will sort it out. We rely on the existing determination of if metacharacters mean it needs to run under cmd and if it needs the ^ quoting which seems to be well tested, and which all runs after this does. Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* Update CHANGELOG.md with details from pull request #179Chef Expeditor2019-04-231-4/+4
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Merge pull request #179 from chef/boilerplateTim Smith2019-04-2310-11/+96
|\ | | | | Add new github templates and codeowners file
| * Add new github templates and codeowners fileTim Smith2019-04-2310-11/+96
|/ | | | | | This syncs our config here with our other repos Signed-off-by: Tim Smith <tsmith@chef.io>
* Fix chefstyle warningsTim Smith2019-04-191-2/+2
| | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* Update expeditor config to use the new labelsTim Smith2019-04-191-2/+2
| | | | | | We updated all our labels Signed-off-by: Tim Smith <tsmith@chef.io>
* Bump version to 3.0.1 by Chef Expeditorv3.0.1Chef Expeditor2019-04-193-4/+6
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Merge pull request #178 from martinisoft/martinisoft/fix_readmeTim Smith2019-04-191-1/+1
|\ | | | | Misnamed parameter in README
| * Misnamed parameterAaron Kalin2019-04-191-1/+1
|/ | | | Signed-off-by: Aaron Kalin <akalin@martinisoftware.com>
* Bump version to 3.0.0 by Chef Expeditorv3.0.0Chef Expeditor2019-01-231-2/+10
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Merge pull request #176 from chef/lcg/version-bump2Tim Smith2019-01-224-9/+11
|\ | | | | update travis/appveyor, drop ruby 2.2 support, test on 2.6
| * update travis/appveyor, drop ruby 2.2 support, test on 2.6lcg/version-bump2Lamont Granquist2019-01-044-9/+11
|/ | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* Update CHANGELOG.md to reflect the promotion of 2.4.4Chef Expeditor2018-12-121-10/+7
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Bump version to 2.4.4 by Chef Expeditorv2.4.4Chef Expeditor2018-12-123-5/+6
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Merge pull request #173 from chef/slim2Tim Smith2018-12-111-1/+0
|\ | | | | Don't ship the readme in the gem artifact
| * Don't ship the readme in the gem artifactTim Smith2018-12-111-1/+0
| | | | | | | | | | | | There's no need for this. Signed-off-by: Tim Smith <tsmith@chef.io>
* | Bump version to 2.4.3 by Chef Expeditorv2.4.3Chef Expeditor2018-12-073-4/+12
| | | | | | | | Obvious fix; these changes are the result of automation not creative thinking.
* | Merge pull request #172 from chef/promote_windowsLamont Granquist2018-12-072-0/+1
|\ \ | |/ | | Have expeditor promote the windows gem as well
| * Have expeditor promote the windows gem as wellpromote_windowsTim Smith2018-12-062-0/+1
|/ | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* Update CHANGELOG.md to reflect the promotion of 2.4.2Chef Expeditor2018-12-061-10/+7
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Bump version to 2.4.2 by Chef Expeditorv2.4.2Chef Expeditor2018-12-063-5/+6
| | | | Obvious fix; these changes are the result of automation not creative thinking.
* Merge pull request #171 from chef/dev_depsTim Smith2018-12-062-5/+2
|\ | | | | Remove dev deps from the gemspec
| * Remove dev deps from the gemspecTim Smith2018-12-062-5/+2
| | | | | | | | | | | | These are all in the gemfile now Signed-off-by: Tim Smith <tsmith@chef.io>
* | Bump version to 2.4.1 by Chef Expeditorv2.4.1Chef Expeditor2018-12-063-5/+13
| | | | | | | | Obvious fix; these changes are the result of automation not creative thinking.
* | Merge pull request #170 from chef/appveyorTim Smith2018-12-064-15/+54
|\ \ | |/ | | Test on ruby-head and Ruby 2.6 in Travis
| * Test on ruby-head and 2.6 in TravisTim Smith2018-12-064-15/+54
| | | | | | | | | | | | Also skip the tests when expeditor commits. Signed-off-by: Tim Smith <tsmith@chef.io>
* | Add code of conduct fileTim Smith2018-12-061-0/+1
|/ | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* Merge pull request #169 from chef/expeditorTim Smith2018-12-025-0/+83
|\ | | | | Wire up Expeditor
| * Wire up ExpeditorTim Smith2018-12-025-0/+83
|/ | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* Merge pull request #167 from chef/slimTim Smith2018-10-234-11/+9
|\ | | | | Don't ship the gemfile/rakefile/gemspecs in the gem
| * Don't ship rakefile/gemfile/gemspecsslimTim Smith2018-10-231-1/+1
| | | | | | | | | | | | We don't ship the specs. We're not running the tests on this gem. Signed-off-by: Tim Smith <tsmith@chef.io>
| * Remove bin / executable defsTim Smith2018-10-231-2/+0
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Provide a more useful homepageTim Smith2018-10-231-1/+1
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Test on Ruby 2.5.3Tim Smith2018-10-231-1/+1
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
| * Update the github links in commentsTim Smith2018-10-232-6/+6
|/ | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* Merge pull request #155 from zerikv/fix-win-cmd-lead-spaceTyler Ball2018-08-201-4/+3
|\ | | | | Avoid EmptyWindowsCommand error on Windows when the command starts with spaces
| * Fix EmptyWindowsCommand error on Windows when the command starts with spacesEric VERNIER2018-07-301-4/+3
| | | | | | | | Signed-off-by: Eric Vernier <zerikv@gmail.com>
* | Merge pull request #166 from chef/githubTim Smith2018-08-012-1/+18
|\ \ | |/ |/| Update codeowners and add github PR template
| * Update codeowners and add github PR templategithubTim Smith2018-07-312-1/+18
|/ | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* Merge pull request #164 from chef/new_rubyv2.4.0Tim Smith2018-07-232-33/+28
|\ | | | | Release 2.4.0
| * Release 2.4.0new_rubyTim Smith2018-07-232-33/+28
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* | Merge pull request #163 from chef/new_rubyTim Smith2018-07-231-3/+4
|\ \ | |/ | | Test on the latest ruby releases
| * Test on the latest ruby releasesTim Smith2018-07-231-3/+4
| | | | | | | | Signed-off-by: Tim Smith <tsmith@chef.io>
* | Merge pull request #158 from MsysTechnologiesllc/dh/MSYS-847_fix_bare_rescuesTim Smith2018-07-233-7/+3
|\ \ | |/ |/| Fix a bare rescue on Windows
| * [MSYS-847] fix bare rescuedheerajd-msys2018-07-033-7/+3
| | | | | | | | Signed-off-by: dheerajd-msys <dheeraj.dubey@msystechnologies.com>
* | Merge pull request #161 from chef/lcg/chefstyle2Lamont Granquist2018-07-064-9/+10
|\ \ | |/ |/| master of chefstyle + fixes
| * master of chefstyle + fixesLamont Granquist2018-07-064-9/+10
|/ | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* Merge pull request #159 from chef/lcg/remove-hashrocketsLamont Granquist2018-06-266-47/+50
|\ | | | | remove hashrocket syntax
| * remove hashrocket syntaxlcg/remove-hashrocketsLamont Granquist2018-06-256-47/+50
|/ | | | Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
* Merge pull request #157 from MsysTechnologiesllc/dh/add_sensitive_featureThom May2018-06-155-14/+25
|\ | | | | [MSYS-830] Added sensitive feature