Athens: Remove -mod=vendor from source and docs

Created on 24 May 2019  路  9Comments  路  Source: gomods/athens

https://github.com/gomods/athens/pull/1231 removed the vendor directory but there are still some -mod=vendor left in our Makefile and docs.

We should remove those and use GOPROXY=https://proxy.golang.org instead to build Athens.

good first issue

Most helpful comment

Let us know if you have any questions.

All 9 comments

Hi! New contributor here. :) I would like to work on this.

Please do @penthaapatel It's yours!

Let us know if you have any questions.

Should I change all the occurrences that say "GOPROXY=http://127.0.0.1:3000" to "GOPROXY=https://proxy.golang.org" in docs/content?

@penthaapatel not everywhere, no. Many of the places in docs that say GOPROXY=http://127.0.0.1:3000 are instructions for how to configure your machine to use an installed Athens, so don't change that. Only change the occurrences that talk about how to build Athens itself. Lots of them will be in development and testing docs

@arschles Thanks! I got it.

I removed -mod=vendor from Makefile, docs/content/install/build-from-source.md, and docs/content/walkthrough.md. Am I doing this right?

@penthaapatel sounds like it! Would you be able to submit this as a pull request and we can start giving feedback directly on the code? Or, did you already submit one and I missed it?

@arschles Okay, I'll submit a PR now. :)

Was this page helpful?
0 / 5 - 0 ratings

Related issues

arschles picture arschles  路  3Comments

opinionsDazzle picture opinionsDazzle  路  4Comments

robjloranger picture robjloranger  路  4Comments

marpio picture marpio  路  3Comments

fedepaol picture fedepaol  路  4Comments