Closed praveenkumar closed 7 years ago
cc @zchee @dlorenc @r2d4
@praveenkumar I have short tested this pull request. LGTM.
In particular, included jceel/lib9p
is great :)
I'll more testing.
@praveenkumar Ah, vendor/lib9p
submodule is duplicate.
Could you try below? (But sorry, I do not know this is the correct way to delete git submodule)
git submodule deinit vendor/lib9p
git rm --cached vendor/lib9p
git rm .gitmodule
git add .gitmodule vendor/lib9p
.
.
@zchee will check and let you know.
@zchee removed lib9p submodule now.
@praveenkumar I'll check it. Thanks.
@praveenkumar LGTM. Thanks :D
This will add glide as dependency management tool which will make easy to have 3rd party modules part of this repo. I also added step in README file. Give it a try.