Tame the deadcode gate and remove dead web helpers

Run deadcode with -test and without the vendored node_modules package so the
check reports only genuinely unreachable code instead of test-only helpers and
vendored functions. Remove the three web helpers it still flagged (formatBikeType,
formatRideType, dataQualityURL) that had no callers. Apply the go fix
modernizations the check gate performs (range-over-int loops and embedded-field
composite literals) that the gate had rewritten in the working tree.

1 files changed, 0 insertions, 14 deletions

diff --git a/web/categories.go b/web/categories.go
index 4077092..a76fa1c 100644
--- a/web/categories.go
+++ b/web/categories.go
@@ -67,20 +67,6 @@ var rideTypeLabels = map[ride.RideType]string{
 	ride.RideTypeTransport: "Transport",
 }
 
-func formatBikeType(value ride.BikeType) string {
-	if label, ok := bikeTypeLabels[value]; ok {
-		return label
-	}
-	return string(value)
-}
-
-func formatRideType(value ride.RideType) string {
-	if label, ok := rideTypeLabels[value]; ok {
-		return label
-	}
-	return string(value)
-}
-
 func formatCategoryOrEmpty(value string) string {
 	if value == "" {
 		return "—"