aboutsummaryrefslogtreecommitdiff
path: root/internal/fetch/fetchdata_test.go
AgeCommit message (Collapse)Author
2025-10-23all: fix some commentscuishuang
Change-Id: Ia70b35ab9aef270692a3892629c3c81208771089 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/713360 Reviewed-by: Dmitri Shuralyov <dmitshur@google.com> Reviewed-by: Sean Liao <sean@liao.dev> kokoro-CI: kokoro <noreply+kokoro@google.com> Auto-Submit: Sean Liao <sean@liao.dev> LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> Reviewed-by: David Chase <drchase@google.com>
2024-12-05static/frontend: add titles to links for visual usersDenis Bowen
frontend: add titles to links add titles to links for visual users Screenshot of fix: https://screenshot.googleplex.com/8dUyg5GVTn7iXmg fixes b/283533047 Change-Id: I2e75c7c6b32add6f42de3a84de34f3219cad935a GitHub-Last-Rev: 4832e52fa3021cac5fdfb38261ea1e83a154d70d GitHub-Pull-Request: golang/pkgsite#101 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/624236 TryBot-Bypass: Jonathan Amsterdam <jba@google.com> Reviewed-by: Nooras Saba‎ <saba@golang.org> Reviewed-by: Jonathan Amsterdam <jba@google.com> Reviewed-by: Cherry Mui <cherryyz@google.com> LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> Reviewed-by: Joy Serquina (xWF) <serquina@google.com>
2024-02-14internal/fetch: handle error in root packageJonathan Amsterdam
If a module had a problem with the package at its root, we would end up with two packageVersionStates. That caused a DB error in upsertPackageVersionStates. Distinguish between the module unit and package units in a couple of places. That addresses the root cause of the problem. Also, fix an unrelated bug that was causing a panic when we tried to send the error to the errorreporting service. Change-Id: Ia02faea50f4477999c6c6eca0674b8dde0f15bb3 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/564076 Reviewed-by: Michael Matloob <matloob@golang.org> LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> kokoro-CI: kokoro <noreply+kokoro@google.com>
2024-01-23internal: move UnitMeta.IsRedistributable to UnitMichael Matloob
Change-Id: Ia0a18df32d752f20b0400b47860638c7d0910eb7 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/556435 Reviewed-by: Jonathan Amsterdam <jba@google.com> kokoro-CI: kokoro <noreply+kokoro@google.com> LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> Reviewed-by: Robert Findley <rfindley@google.com>
2023-11-08internal/godoc/dochtml: add type parameters on func synopsisThiago
Adding type parameters on functions synopsis. Fixes golang/go#60954 Change-Id: If72c1b4d401b6cd32c335b1ed136063e7c965743 GitHub-Last-Rev: 375797ea2c2b352d6e5435334b82804552fd1a57 GitHub-Pull-Request: golang/pkgsite#79 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/535555 Reviewed-by: Suzy Mueller <suzmue@golang.org> kokoro-CI: kokoro <noreply+kokoro@google.com> LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> Reviewed-by: Jonathan Amsterdam <jba@google.com>
2023-09-05static/frontend: fix to make link purpose clear to users.Prudhvi Krishna Chintala
Change-Id: I4ebdf32f03936e6af927c204e28548e39994cd77 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/520255 LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> TryBot-Result: Gopher Robot <gobot@golang.org> Auto-Submit: Suzy Mueller <suzmue@golang.org> Reviewed-by: Jamal Carvalho <jamal@golang.org> Run-TryBot: Suzy Mueller <suzmue@golang.org> Reviewed-by: Suzy Mueller <suzmue@golang.org> kokoro-CI: kokoro <noreply+kokoro@google.com>
2023-07-17internal/fetch: add two stdlib gettersMichael Matloob
This breaks out the behavior of getting the stdlib into its own getter, stdlibZipModuleGetter. Most (but not all) of the special cases in the fetch code for the stdlib are in the stdlibZipModuleGetter, which is installed on the frontend. For the local pkgsite command, we add a new builder for the goPackagesModuleGetter that loads the packages from the stdlib given a GOROOT. The stdlibZipModuleGetter is also installed for pkgsite in as a fallback if the goPackagesModuleGetter doesn't successfully fetch the stdlib module (I'm not sure if that's possible). Fixes #60114 Change-Id: Ida6a5367343643cc337c6d05e0d40095f79ca9e5 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/507883 Reviewed-by: Jamal Carvalho <jamal@golang.org> TryBot-Result: Gopher Robot <gobot@golang.org> Run-TryBot: Michael Matloob <matloob@golang.org> kokoro-CI: kokoro <noreply+kokoro@google.com>
2022-06-17many: re-enable staticcheck and fix problemsJonathan Amsterdam
Most of the checks were about io/ioutil. There were a couple of other minor ones. I didn't address the check for strings.Title; instead, I turned off that check globally with a staticcheck.conf file. Change-Id: I286a6894fb1fd891818ab9e451c891f52a3828fc Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/412675 Reviewed-by: Jamal Carvalho <jamal@golang.org> Run-TryBot: Jonathan Amsterdam <jba@google.com> TryBot-Result: kokoro <noreply+kokoro@google.com>
2022-04-18internal/fetch: list known alternative module pathsJonathan Amsterdam
Some modules don't have go.mod files. Nonetheless there is clearly a canonical module path, and various forks that should be marked as alternative. A major example is github.com/Azure/azure-sdk-for-go, the canonical path, and forks like gopkg.in/Azure/azure-sdk-for-go. For large modules like the Azure SDK, these forks consume a lot of processing bandwidth. Although we already have a mechanism for this (see largefork.go), it requires both contents and version numbers to match a known list of modules and versions. The list can get out of date, and it doesn't include every possible version. For instance, the gopkg.in Azure fork consists of many pseudo-versions that aren't in the proxy list. This CL adds a list of module paths that are known to be forks. They will be marked as alternative modules regardless of version or contents. Fixes golang/go#52329. Change-Id: Ie374da67f0f42f50be551a1078b667686453b447 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/400357 Run-TryBot: Jonathan Amsterdam <jba@google.com> TryBot-Result: kokoro <noreply+kokoro@google.com> Reviewed-by: Julie Qiu <julieqiu@google.com>
2022-02-04internal/fetch: add generics testJonathan Amsterdam
For golang/go#48264 Change-Id: I94b1bae5b2a59753492148edf2e1f2931235802f Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/382976 Trust: Jonathan Amsterdam <jba@google.com> Run-TryBot: Jonathan Amsterdam <jba@google.com> Reviewed-by: Jamal Carvalho <jamal@golang.org> TryBot-Result: kokoro <noreply+kokoro@google.com>
2021-08-24internal/fetch: populate Unit.BuildContextsJonathan Amsterdam
Populate the lists of build contexts when fetching a module. Currently we only do this when reading from the DB. We want DataSources that rely only on fetch to have the right value. For golang/go#47780 Change-Id: Ie85fba8ccb318bb4ed5c1f2153b0ba4216358022 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/344670 Trust: Jonathan Amsterdam <jba@google.com> Run-TryBot: Jonathan Amsterdam <jba@google.com> TryBot-Result: kokoro <noreply+kokoro@google.com> Reviewed-by: Julie Qiu <julie@golang.org>
2021-08-18internal/proxy/proxytest: proxy test codeJonathan Amsterdam
Most of the code in internal/proxy was only for testing. Move it to a separate package for clarity. Change-Id: Ie9d241bc8a12923b579a7dd69ae7b36aaa1fd42a Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/343209 Trust: Jonathan Amsterdam <jba@google.com> Run-TryBot: Jonathan Amsterdam <jba@google.com> TryBot-Result: kokoro <noreply+kokoro@google.com> Reviewed-by: Julie Qiu <julie@golang.org>
2021-07-16internal/godoc/dochtml: change formatting for structs and interfacesJulie Qiu
Previously, the trailing curly braces in the struct or interface synopsis were removed before inserting into the database, since they are not necessary for displaying symbol history. However, this information is useful in symbol search results to indicate fields and methods, so they are now added back. For symbol history, they are removed on the frontend. For golang/go#44142 Change-Id: Iaf4f98c73834d92ac944a53a078a1b38adb2e11b Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/335069 Trust: Julie Qiu <julie@golang.org> Run-TryBot: Julie Qiu <julie@golang.org> TryBot-Result: kokoro <noreply+kokoro@google.com> Reviewed-by: Jamal Carvalho <jamal@golang.org> Reviewed-by: Jonathan Amsterdam <jba@google.com>
2021-07-12internal: support std@dev.fuzzJulie Qiu
It is now possible to fetch std@dev.fuzz and view /std@dev.fuzz on the frontend. For golang/go#46910 Change-Id: I2fd7259840ba32ff35ca323d07412988e533ffba Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/330413 Trust: Julie Qiu <julie@golang.org> Run-TryBot: Julie Qiu <julie@golang.org> TryBot-Result: kokoro <noreply+kokoro@google.com> Reviewed-by: Jonathan Amsterdam <jba@google.com>
2021-05-04internal: change Symbol.Children to SymbolMetaJulie Qiu
The children of a symbol will always have the same build context and no other children. As a result, Symbol.Children are changed to the SymbolMeta type. For golang/go#37102 Change-Id: I89c361f86eff7179c18572e57a5dd0788fea4b1d Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/315432 Trust: Julie Qiu <julie@golang.org> Run-TryBot: Julie Qiu <julie@golang.org> TryBot-Result: kokoro <noreply+kokoro@google.com> Reviewed-by: Jonathan Amsterdam <jba@google.com>
2021-05-04internal: factor out SymbolMetaJulie Qiu
UnitSymbol and Symbol share a lot of fields, which are now factored out as SymbolMeta. For golang/go#37102 Change-Id: Ie8590b0a50dafa4199fbe165d94f28028d1a0bb5 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/315431 Trust: Julie Qiu <julie@golang.org> Run-TryBot: Julie Qiu <julie@golang.org> TryBot-Result: kokoro <noreply+kokoro@google.com> Reviewed-by: Jonathan Amsterdam <jba@google.com>
2021-04-21internal/godoc: don't render doc on workerJonathan Amsterdam
The Package.Render method is only used on the worker to get the synopsis and other doc-related information. Ever since we started rendering doc on the frontend, the worker has ignored the rendered documentation. So stop rendering the doc, and rename the method to DocInfo to reflect that. One minor consequence is that we no longer flag packages with excessively large doc as having a bad status. As far as the worker is concerned, they are fine; the error will manifest on the frontend (and we will serve a "documentation too large" page). This is all good: we only used a distinct status in this case so we could reprocess if our limits changed, but these modules no longer need reprocessing. For golang/go#40850 Change-Id: I3c7c49f0beb7a6d8a37daabdce75f83ef108eddb Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/312270 Trust: Jonathan Amsterdam <jba@google.com> Run-TryBot: Jonathan Amsterdam <jba@google.com> TryBot-Result: kokoro <noreply+kokoro@google.com> Reviewed-by: Jamal Carvalho <jamal@golang.org> Reviewed-by: Julie Qiu <julie@golang.org>
2021-03-17internal/fetch: don't fail on bad build contextJonathan Amsterdam
If one build context of a package is bad (say, because it defines two different package names), don't reject the entire package. Just keep the valid build contexts. Fixes golang/go#45067 Change-Id: I3128473e2350d4f04b353bb53cbd2f17bdb39bfc Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/302534 Trust: Jonathan Amsterdam <jba@google.com> Run-TryBot: Jonathan Amsterdam <jba@google.com> Reviewed-by: Julie Qiu <julie@golang.org>
2021-03-15internal: change synopsis stored in databaseJulie Qiu
Now that the unique constraint on package_symbols has changed, the synopsis that is presented on pkg.go.dev can be stored in the database, instead of a version that is unique to each symbol. For golang/go#37102 Change-Id: I1fe548ee548a4db7eee27fb1a89fcca1afc7eecf Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/301109 Trust: Julie Qiu <julie@golang.org> Run-TryBot: Julie Qiu <julie@golang.org> TryBot-Result: kokoro <noreply+kokoro@google.com> Reviewed-by: Jonathan Amsterdam <jba@google.com>
2021-03-02internal/fetch: determine go.mod presence more oftenJonathan Amsterdam
Record whether a module has a go.mod file in more cases (as long as we can read the zip), even if the module itself is not good in some way. Change-Id: I8c22ae8325d60d5b690f9aa9a0d2852aa3f9001f Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/296809 Trust: Jonathan Amsterdam <jba@google.com> Run-TryBot: Jonathan Amsterdam <jba@google.com> TryBot-Result: kokoro <noreply+kokoro@google.com> Reviewed-by: Julie Qiu <julie@golang.org>
2021-03-01internal/godoc/dochtml: fix synopsis for symbolsJulie Qiu
The synopsis for symbols are updated such that: * Constants are always represented as const <name>, so that we do not need to handle evaluating them in multi-line blocks. * Bugs in synopsis for multi-variable blocks are fixed * Field synopsis on structs are now represented as type <StructName> struct, <FieldName> <Type> For example: type Foo struct, Bar string * Method synopsis on interfaces are now represented as: type <InterfaceName> interface, <Method Signature> For example: type Foo interface, Bar() string Change-Id: I96f219cfd398c47c9a5be23fe4c01802efd98aaa Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/296631 Trust: Julie Qiu <julie@golang.org> Reviewed-by: Jamal Carvalho <jamal@golang.org> Reviewed-by: Jonathan Amsterdam <jba@google.com>
2021-03-01internal: do not display README at /stdJulie Qiu
The README is no longer displayed at pkg.go.dev/std. It is also no longer stored in the database when a std module is fetched. For golang/go#44356 Change-Id: I6b1605389bc3c97694b6ec2b06e28006a38be999 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/296950 Trust: Julie Qiu <julie@golang.org> Reviewed-by: Jonathan Amsterdam <jba@google.com>
2021-02-25internal: embed ModuleInfo into UnitMetaJonathan Amsterdam
UnitMeta had most of the fields of ModuleInfo, but did not embed it. Now that we have added four more fields to ModuleInfo for deprecation and retraction, it makes sense to embed it rather than duplicate those fields. For golang/go#43265 For golang/go#41321 Change-Id: I20e2b922b49c7873a5535745d644631123de37cd Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/296209 Trust: Jonathan Amsterdam <jba@google.com> Run-TryBot: Jonathan Amsterdam <jba@google.com> TryBot-Result: kokoro <noreply+kokoro@google.com> Reviewed-by: Julie Qiu <julie@golang.org>
2021-02-22internal/stdlib: fix Zip for master versionJulie Qiu
An error is fixed when creating the zipfile for std@master, which prevented the module from being fetched. A test is also added. For golang/go#44390 Change-Id: I279146aef10311822f47f16259a2223c879140c2 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/294831 Trust: Julie Qiu <julie@golang.org> Run-TryBot: Julie Qiu <julie@golang.org> Reviewed-by: Jonathan Amsterdam <jba@google.com>
2021-02-17internal/godoc/dochtml: update synopsis for consts and varsJulie Qiu
The synopsis for constants and variables is updated to include the value of that symbol. For golang/go#37102 Change-Id: I181704ef9bd586bc586cecb69f32927b13f5dd68 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/291790 Trust: Julie Qiu <julie@golang.org> Run-TryBot: Julie Qiu <julie@golang.org> Reviewed-by: Jonathan Amsterdam <jba@google.com>
2021-02-11internal/fetch: use standard test modules where feasibleJonathan Amsterdam
Replace some of the more straightforward modules for testing FetchModule with standard test modules or variants. Leave most of the specialized test modules as they are. For golang/go#44214 Change-Id: I014c20cfe9f1c8cf75770a5f7f714357b2db7fe9 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/290897 Trust: Jonathan Amsterdam <jba@google.com> Run-TryBot: Jonathan Amsterdam <jba@google.com> TryBot-Result: kokoro <noreply+kokoro@google.com> Reviewed-by: Julie Qiu <julie@golang.org>
2021-02-10internal/godoc/{dochtml}: add GetSymbolsJulie Qiu
A GetSymbols function is added to packages godoc and dochtml, which together return the symbols in the package API for a given documentation source. For golang/go#37102 Change-Id: Iab54a7f50101b87ed0289509c1d0d6e439f5d43c Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/289752 Trust: Julie Qiu <julie@golang.org> Run-TryBot: Julie Qiu <julie@golang.org> TryBot-Result: kokoro <noreply+kokoro@google.com> Reviewed-by: Jonathan Amsterdam <jba@google.com>
2021-02-08internal/fetch: collapse identical build contextsJonathan Amsterdam
If all the build contexts for a package produce the same documentation, then return a single Documentation with build context all/all. This will save a lot of space in the documentation table, since most packages will fit into this case. For golang/go#37232 Change-Id: I15237242e0c3ca3c7b8f8c8944b227afebd23785 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/289680 Trust: Jonathan Amsterdam <jba@google.com> Run-TryBot: Jonathan Amsterdam <jba@google.com> TryBot-Result: kokoro <noreply+kokoro@google.com> Reviewed-by: Julie Qiu <julie@golang.org>
2021-02-08internal: support "all" in BuildContextJonathan Amsterdam
Handle BuildContexts where one or more parts are "all". These should only occur when searching for a matching Documentation, not sorting. (Because we sort only when there is more than one Documentation, and if there is an all/all Documentation then there aren't any others.) For golang/go#37232 Change-Id: I898aba8f73d2682798d56c47cc6773709f10c702 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/290094 Trust: Jonathan Amsterdam <jba@google.com> Run-TryBot: Jonathan Amsterdam <jba@google.com> TryBot-Result: kokoro <noreply+kokoro@google.com> Reviewed-by: Julie Qiu <julie@golang.org>
2021-02-05Revert "internal: support "all" in BuildContext"Jonathan Amsterdam
This reverts commit 5289fe63bf336e52e579b0e30c4616bcfe3a31ad. Reason for revert: breaks build Change-Id: Iedc647645373a2c3e1ddf6a414564edfc4cae406 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/290069 Reviewed-by: Julie Qiu <julie@golang.org> Trust: Jonathan Amsterdam <jba@google.com>
2021-02-05internal: support "all" in BuildContextJonathan Amsterdam
Handle BuildContexts where one or more parts are "all". These should only occur when searching for a matching Documentation, not sorting. (Because we sort only when there is more than one Documentation, and if there is an all/all Documentation then there aren't any others.) Change-Id: Ibe28bd78fa278c042d1459f7046bfe8eef49bbac Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/289679 Trust: Jonathan Amsterdam <jba@google.com> Run-TryBot: Jonathan Amsterdam <jba@google.com> Reviewed-by: Julie Qiu <julie@golang.org>
2021-02-02many: allow for multiple documentations for a UnitJonathan Amsterdam
Change internal.Unit.Documentation from a single internal.Documentation to a slice, so that a Unit can have doc for multiple build contexts. However, don't use this ability yet: fetch takes only the first build context it finds, postgres inserts only the first, frontend renders only the first, etc. For golang/go#37232 Change-Id: I17fd1ff2378e0862c029c8077057c207001f6136 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/288217 Trust: Jonathan Amsterdam <jba@google.com> Run-TryBot: Jonathan Amsterdam <jba@google.com> TryBot-Result: kokoro <noreply+kokoro@google.com> Reviewed-by: Julie Qiu <julie@golang.org> Reviewed-by: Jamal Carvalho <jamal@golang.org>
2021-01-15internal/fetch: reduce max doc sizeJonathan Amsterdam
The test that reads excessively large documentation keeps timing out in CI. Try halving the size of the doc. Change-Id: I298f8ff2522937f7129bd3d968cf498ac7efbd56 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/284233 Trust: Jonathan Amsterdam <jba@google.com> Run-TryBot: Jonathan Amsterdam <jba@google.com> TryBot-Result: kokoro <noreply+kokoro@google.com> Reviewed-by: Julie Qiu <julie@golang.org>
2020-12-15internal/source,stdlib: change stdlib source repo to go.googlesource.comJonathan Amsterdam
Link to source files in the canonical Go repo, go.googlesource.com/go. Continue to serve raw content (for images) from the GitHub mirror, since we can't from googlesource.com. Fixes golang/go#43180 Change-Id: Iac513d6481bcdb73ae73eed40d33518ffd3a0158 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/278293 Trust: Jonathan Amsterdam <jba@google.com> Run-TryBot: Jonathan Amsterdam <jba@google.com> Reviewed-by: Julie Qiu <julie@golang.org> TryBot-Result: kokoro <noreply+kokoro@google.com>
2020-11-24internal/{godoc,worker,fetch}: factor out doc rendering for testsJonathan Amsterdam
Add godoc.RenderPartsFromUnit to simplifying doc rendering for tests, add use it in internal/{worker,fetch}. Also remove other unused symbols. Change-Id: Ifeef2cf6088332a3e4496be47a86afde885ecc54 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/272790 Trust: Jonathan Amsterdam <jba@google.com> Run-TryBot: Jonathan Amsterdam <jba@google.com> TryBot-Result: kokoro <noreply+kokoro@google.com> Reviewed-by: Julie Qiu <julie@golang.org>
2020-11-24many: remove Documentation.HTMLJonathan Amsterdam
HTML is always generated from source, so the Documentation.HTML field is no longer necessary. Change-Id: I13cddbfbd983c8a2ff0cb9389e854887caadcc13 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/272509 Trust: Jonathan Amsterdam <jba@google.com> Run-TryBot: Jonathan Amsterdam <jba@google.com> Reviewed-by: Julie Qiu <julie@golang.org> TryBot-Result: kokoro <noreply+kokoro@google.com>
2020-10-06internal/godoc/dochtml: stop adding /pkg prefix to linksJonathan Amsterdam
This prefix is removed when reading from the DB, but not after a frontend render. I think we can just stop adding it. Caught by an improved integration test which now diffs the worker and frontend docs, also included. Change-Id: I835f18ca5aa602700ad6d3ce34bc57e56c60f2d1 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/259101 Trust: Jonathan Amsterdam <jba@google.com> Run-TryBot: Jonathan Amsterdam <jba@google.com> TryBot-Result: kokoro <noreply+kokoro@google.com> Reviewed-by: Julie Qiu <julie@golang.org>
2020-10-02internal: deprecate LegacyModuleInfoJulie Qiu
For golang/go#39629 Change-Id: I157a325cfb4be06d77860ef5eb77b7bdb15a532a Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/258800 Trust: Julie Qiu <julie@golang.org> Run-TryBot: Julie Qiu <julie@golang.org> TryBot-Result: kokoro <noreply+kokoro@google.com> Reviewed-by: Jonathan Amsterdam <jba@google.com>
2020-10-01internal/fetch: do not return legacy readme data for FetchModuleJulie Qiu
fetch.FetchModule no longer returns LegacyReadmeFilePath and LegacyReadmeContents. For golang/go#39629 Change-Id: Iafd33c45ead827a0d4eb6a5f0733bb32ce9a8acf Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/258566 Trust: Julie Qiu <julie@golang.org> Run-TryBot: Julie Qiu <julie@golang.org> TryBot-Result: kokoro <noreply+kokoro@google.com> Reviewed-by: Jonathan Amsterdam <jba@google.com>
2020-09-02internal: remove Unit.PackageJulie Qiu
Unit.Pacakge is removed, since it isn't need for anything. For golang/go#39629 Change-Id: Ic620603550874de8a0b4c37c67a88bee65ec7933 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/252323 Reviewed-by: Jonathan Amsterdam <jba@google.com>
2020-09-02internal: move Unit.Package.Documentation to Unit.DocumentationJulie Qiu
The Documentation field is moved from Unit.Package.Documentation to Unit.Documentation. Unit.Package will be deprecated in a later CL. For golang/go#39629 Change-Id: Idc6d6598a2647a55b55c947120c322fec5da59cc Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/252321 Reviewed-by: Jonathan Amsterdam <jba@google.com>
2020-09-01internal: rename PathInfo to UnitMetaJulie Qiu
PathInfo is renamed to UnitMeta, and GetPathInfo is renamed to GetUnitMeta. For golang/go#39629 Change-Id: Ia91cf7d2752988653efcc0e46a4d6b8b63017ef3 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/251978 Run-TryBot: Julie Qiu <julie@golang.org> TryBot-Result: kokoro <noreply+kokoro@google.com> Reviewed-by: Jonathan Amsterdam <jba@google.com>
2020-09-01internal: replace Unit.DirectoryMeta with Unit.PathInfoJulie Qiu
Unit.DirectoryMeta is replaced with Unit.PathInfo DirectoryMeta isn't necessary and will be deleted in a future CL. For golang/go#39629 Change-Id: I521a062207d3f48592d6e949306a015458d1398d Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/251957 Run-TryBot: Julie Qiu <julie@golang.org> TryBot-Result: kokoro <noreply+kokoro@google.com> Reviewed-by: Jonathan Amsterdam <jba@google.com>
2020-08-31internal: delete DirectoryMeta.V1PathJulie Qiu
DirectoryMeta.V1Path isn't used anywhere, so it is deleted and computed using internal.V1Path when needed For golang/go#39629 Change-Id: I6c69bf098ec0aff45b830a1400885b54d50937ef Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/251580 Run-TryBot: Julie Qiu <julie@golang.org> TryBot-Result: kokoro <noreply+kokoro@google.com> Reviewed-by: Jonathan Amsterdam <jba@google.com>
2020-08-31internal: remove VersionType from ModuleInfoJulie Qiu
ModuleInfo.VersionType isn't being used, except to insert the field in InsertModule. The version type is now calculated in that function and removed from ModuleInfo. For golang/go#39629 Change-Id: I42e13bcfe9540a1b11babea8444cd053fc4db0c6 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/251698 Run-TryBot: Julie Qiu <julie@golang.org> TryBot-Result: kokoro <noreply+kokoro@google.com> Reviewed-by: Jonathan Amsterdam <jba@google.com>
2020-08-28internal: rename Module.Directories to Module.UnitsJulie Qiu
For golang/go#39629 Change-Id: I52d071d035c66dd385e4d32c08b039fddfcb01d7 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/251173 Reviewed-by: Jonathan Amsterdam <jba@google.com>
2020-08-28internal: rename Directory to UnitJulie Qiu
For golang/go#39629 Change-Id: I7abe21764a58b6295926c6cd56ab327824570e77 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/251163 Reviewed-by: Jonathan Amsterdam <jba@google.com>
2020-08-28internal/postgres: add WithImports fieldset to GetDirectoryJulie Qiu
GetDirectory now supports the WithImports fieldset, which lets the caller fetch only the imports. The Package.Imports field is moved to Directory.Imports. For golang/go#39629. Change-Id: I5d597555fa372cc1ee70390460d349fa83f31425 Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/251272 Reviewed-by: Jamal Carvalho <jamal@golang.org>
2020-08-25internal: rename VersionedDirectory to DirectoryJulie Qiu
For golang/go#39629 Change-Id: I17171838c864c1708eab527caa2288c57b6b4def Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/250506 Reviewed-by: Jamal Carvalho <jamal@golang.org>
2020-08-25internal: merge Directory and VersionedDirectoryJulie Qiu
Directory and VersionedDirectory are merged into a single struct, since VersionedDirectory doesn't provide additional value. In this CL, Directory is removed. VersionedDirectory will be renamed to Directory in the next CL to make it easier to review. For golang/go#39629 Change-Id: I4f29bcd5db673c1c715cdd0bc8bd5391cd65d45d Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/250505 Reviewed-by: Jamal Carvalho <jamal@golang.org>
generated by cgit v1.3 (git 2.53.0) at 2026-04-15 12:44:48 +0000