Skip to content
This repository was archived by the owner on Aug 25, 2018. It is now read-only.

Prototype for 'purge' command (#16)#94

Open
excavador wants to merge 1 commit into
FiloSottile:masterfrom
excavador:master
Open

Prototype for 'purge' command (#16)#94
excavador wants to merge 1 commit into
FiloSottile:masterfrom
excavador:master

Conversation

@excavador

@excavador excavador commented Jul 30, 2017

Copy link
Copy Markdown

I prototyped 'purge' command, for that I copied & fixed related file from gb-vendor.
I am not sure how exactly 'ParseImports' should be used for this tasks.
Hope for your help to complete the task

@slomek

slomek commented Aug 3, 2017

Copy link
Copy Markdown

@excavador I guess you need to add pkg/errors to vendor, then the Travis checks may pass.

@excavador

Copy link
Copy Markdown
Author

@FiloSottile could you please provide any feedback how exactly it should be fixed?

@slomek

slomek commented Aug 3, 2017

Copy link
Copy Markdown

@excavador gvt fetch github.com/pkg/errors on your branch and add all updated files.

@excavador

excavador commented Aug 3, 2017

Copy link
Copy Markdown
Author

@slomek I do not sure what that's the right solution (if you compare with other files you''ll pkg/errors used nowhere).
That's why I am asking the owner of the repo, I need him opinion about the "right" way how he see it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants