mdebray/tvl-depot
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

docs(tvix/nar-bridge): remove redundant comments

Browse source 
These really only describe what the code is doing, remove.

Pointed out in
https://cl.tvl.fyi/c/depot/+/9652/comment/987b3b7f_c7b5e47b/

Change-Id: I512a274295244f96744a489871861fed9523bb37
Reviewed-on: https://cl.tvl.fyi/c/depot/+/9654
Reviewed-by: Brian McGee <brian@bmcgee.ie>
Tested-by: BuildkiteCI
This commit is contained in:
Florian Klink 2023-10-11 14:00:24 +02:00 committed by flokli
parent 155ab77cff
commit 6dab608198
1 changed files with 0 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
tvix/nar-bridge/pkg/http/narinfo_get.go
View file

@ -53,14 +53,12 @@ func renderNarinfo(
log = log.WithField("pathInfo", pathInfo) log = log.WithField("pathInfo", pathInfo)
// The PathInfo received needs to be valid, and contain a NARInfo field.
if _, err := pathInfo.Validate(); err != nil { if _, err := pathInfo.Validate(); err != nil {
log.WithError(err).Error("unable to validate PathInfo") log.WithError(err).Error("unable to validate PathInfo")
return fmt.Errorf("unable to validate PathInfo: %w", err) return fmt.Errorf("unable to validate PathInfo: %w", err)
} }
// Ensure the PathInfo contains a NARInfo field
if pathInfo.GetNarinfo() == nil { if pathInfo.GetNarinfo() == nil {
log.Error("PathInfo doesn't contain Narinfo field") log.Error("PathInfo doesn't contain Narinfo field")