[pkgsite] internal: conditionally return documentation in GetUnit

0 views
Skip to first unread message

Gopher Robot (Gerrit)

unread,
1:16 PM (7 hours ago) 1:16 PM
to Ethan Lee, goph...@pubsubhelper.golang.org, golang-...@googlegroups.com, Jonathan Amsterdam, kokoro, Go LUCI, golang-co...@googlegroups.com

Gopher Robot submitted the change

Change information

Commit message:
internal: conditionally return documentation in GetUnit

- Large documentations blobs should avoid being returned unless
explicitly requested. Add a WithDocsSource to enable retrieval of
unit information without the large documentation source information.
- This does not functionally change existing GetUnit callsites.
Change-Id: Ie983b7eaeb61bb92f2716ae64a3d1e1ea738dbe2
Reviewed-by: Jonathan Amsterdam <j...@google.com>
kokoro-CI: kokoro <noreply...@google.com>
Auto-Submit: Ethan Lee <etha...@google.com>
Files:
  • M internal/api/api.go
  • M internal/fetchdatasource/fetchdatasource_test.go
  • M internal/frontend/main.go
  • M internal/postgres/unit.go
  • M internal/postgres/unit_test.go
  • M internal/testing/fakedatasource/fakedatasource.go
  • M internal/unit.go
  • M internal/worker/fetch_test.go
  • M internal/worker/fetcherror_test.go
  • M internal/worker/refetch_test.go
Change size: M
Delta: 10 files changed, 44 insertions(+), 17 deletions(-)
Branch: refs/heads/master
Submit Requirements:
  • requirement satisfiedCode-Review: +2 by Jonathan Amsterdam
  • requirement satisfiedTryBots-Pass: LUCI-TryBot-Result+1 by Go LUCI
  • requirement satisfiedkokoro-CI-Passes: kokoro-CI+1 by kokoro
Open in Gerrit
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: merged
Gerrit-Project: pkgsite
Gerrit-Branch: master
Gerrit-Change-Id: Ie983b7eaeb61bb92f2716ae64a3d1e1ea738dbe2
Gerrit-Change-Number: 756480
Gerrit-PatchSet: 6
Gerrit-Owner: Ethan Lee <etha...@google.com>
Gerrit-Reviewer: Ethan Lee <etha...@google.com>
Gerrit-Reviewer: Gopher Robot <go...@golang.org>
Gerrit-Reviewer: Jonathan Amsterdam <j...@google.com>
Gerrit-Reviewer: kokoro <noreply...@google.com>
Gerrit-CC: kokoro <noreply...@google.com>
open
diffy
satisfied_requirement
Reply all
Reply to author
Forward
0 new messages