sourcegraph/sourcegraph-public-snapshot

insights: refactor capture group methods for readability and performance

Offen

#33.148 geöffnet am 28.03.2022

Auf GitHub ansehenContributor Guide
 (11 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Go (1.374 Forks)auto 404
backendcode-insightsgood first issue

Repository-Metriken

Stars
 (10.290 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

When a capture group insight is created, we tally up matches per capture group using MatchValues and Counts methods on ComputeMatchContext. We then group them in the method GroupByCaptureMatch.

The current logic uses many nested for loops. The aim of this ticket is to look into simplifying this logic. What we want at the end is a map of unique capture group values to their count.

Contributor Guide

Research-Richtung
Überprüfen Sie die Methoden MatchValues, Counts und GroupByCaptureMatch in der Sourcegraph Codebasis. Identifizieren Sie verschachtelte Schleifen und erkunden Sie Alternativen wie die Verwendung einer Map zur direkten Aggregation von Zählungen oder die Verwendung von Streams zur Reduzierung der Komplexität.
Tech Stack
go
Domain
backend
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Go
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60