Bunchhieng/hnreader

remove vendor folder in repo - use dep instead

mukeshmk opened this issue · 6 comments

Currently, the repo contains the vendor folder. Which need not be tracked by version control, as Gopkg.toml and Gopkg.lock files are used to track your dependencies. So please remove the vendor folder.

The vendor folder will be created with the same version of dependencies as specified in Gopkg.toml, upon running dep init.

fixed it with the PR #48.

Just curious, why would you use dep over official go 1.11 modules? (see: golang/dep#1959 (comment))

I agree this info (in the below-mentioned link) is old, but it is still valid:
This is the major difference as mentioned in the PR aka Doc-FAQ.
There is a subtle difference in what you are trying to achieve, that would be the key difference.

golang/dep#376

Right, but I'm not comparing dep to go get. I'm comparing it to go mod, which versions packages in a go.mod file and creates a go.sum file to summarize dependency installation for reproducible builds.

Are we using go mod here? 🤔

No