Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add vendor #23

Merged
merged 1 commit into from
Sep 4, 2016
Merged

add vendor #23

merged 1 commit into from
Sep 4, 2016

Conversation

AkihiroSuda
Copy link
Contributor

I used govendor, but not sticking to it.
Please let me know if you like other vendoring tool

@cpuguy83
Copy link
Owner

cpuguy83 commented Sep 3, 2016

Thanks, looking at govendor... don't think I like it very much.
https://github.com/LK4D4/govend seems simple, WDYT?

Signed-off-by: Akihiro Suda <suda.akihiro@lab.ntt.co.jp>
@AkihiroSuda
Copy link
Contributor Author

OK, updated PR to use govend

@@ -9,3 +9,10 @@ Uses blackfriday to process markdown into man pages.
### Usage

./md2man -in /path/to/markdownfile.md -out /manfile/output/path

### How to contribute

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I tried to describe govend in vendor/README.md, but govend -v -u --prune automatically removes extra files under the vendor directory.
So I edited the top-level README.md

@cpuguy83
Copy link
Owner

cpuguy83 commented Sep 4, 2016

Awesome, LGTM.

Thanks!

@cpuguy83 cpuguy83 merged commit a65d4d2 into cpuguy83:master Sep 4, 2016
@AkihiroSuda
Copy link
Contributor Author

Thank you for merging, can you please release v1.0.6 if it is ready?
(not urgent 😄 )

@cpuguy83
Copy link
Owner

cpuguy83 commented Sep 7, 2016

@AkihiroSuda Done :)

@AkihiroSuda
Copy link
Contributor Author

Thank you 😃

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants