search

manifoldco / torus-cli

A secure, shared workspace for secrets
https://www.torus.sh
BSD 3-Clause "New" or "Revised" License
613 stars 32 forks source link

Add dep prune options to dep manifest #358

Closed arbourd closed 6 years ago

arbourd commented 6 years ago

With 0.4.0, dep now runs prune automatically when when using ensure. Dep will check in the manifest for what options are needed for pruning.

This commit sets the prune settings to remove non-go files (exlcuding LICENSE and AUTHORS), go test files. Unused (non-imported) packages need to be included because of the tools/ package.

AppVeyorBot commented 6 years ago

:x: Build torus-cli 483 failed (commit https://github.com/manifoldco/torus-cli/commit/ea66c8b504 by @arbourd)

arbourd commented 6 years ago

Looks like the build is failing because this broke a cache. I'm seeing the same errors in master on my fork: https://travis-ci.org/arbourd/torus-cli/builds/333346471

ianlivingstone commented 6 years ago

Looks like the build is failing because this broke a cache. I'm seeing the same errors in master on my fork: https://travis-ci.org/arbourd/torus-cli/builds/333346471

I just cleared the cache and restarted the build for you

AppVeyorBot commented 6 years ago

:x: Build torus-cli 514 failed (commit https://github.com/manifoldco/torus-cli/commit/038b445b1c by @arbourd)

AppVeyorBot commented 6 years ago

:x: Build torus-cli 539 failed (commit https://github.com/manifoldco/torus-cli/commit/0f75e84b8a by @arbourd)

AppVeyorBot commented 6 years ago

:white_check_mark: Build torus-cli 540 completed (commit https://github.com/manifoldco/torus-cli/commit/0835b4837f by @arbourd)

arbourd commented 6 years ago

Build failures were due to the dep prune settings. Removing unused dependencies pruned go-bindata, which is not imported, but needed for the toolchain.

Sorry for all the failures!

ianlivingstone commented 6 years ago

Awesome, thanks for this :)