Skip to content
This repository was archived by the owner on Sep 9, 2020. It is now read-only.

Commit 1963157

Browse files
authored
Merge pull request #1074 from AlekSi/patch-1
Expand FAQ entry
2 parents 59c9596 + 9d480dc commit 1963157

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

docs/FAQ.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -296,13 +296,13 @@ It's up to you:
296296

297297
**Pros**
298298

299-
- it's the only way to get truly reproducible builds, as it guards against upstream renames and deletes
300-
- you don't need an extra `dep ensure` step (to fetch dependencies) on fresh clones to build your repo
299+
- It's the only way to get truly reproducible builds, as it guards against upstream renames, deletes and commit history overwrites.
300+
- You don't need an extra `dep ensure` step (to fetch dependencies) on fresh clones to build your repo: you can use `go get` as usual.
301301

302302
**Cons**
303303

304-
- your repo will be bigger, potentially a lot bigger
305-
- PR diffs are more annoying
304+
- Your repo will be bigger, potentially a lot bigger.
305+
- PR diffs are more annoying (but files in `vendor/` are [hidden by default](https://github.com/github/linguist/blob/v5.2.0/lib/linguist/generated.rb#L328) on Github).
306306

307307
## How do I roll releases that `dep` will be able to use?
308308

0 commit comments

Comments
 (0)