Data.List.NonEmpty.intersect and Data.List.NonEmpty.intersectBy throw when intersection empty

Offen
#216 2 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

@evaogbe arbeitet bereits daran.

Seit 14.12.2023.

  • #217 von @evaogbe — offen

Bewertung

Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Anfängerfreundlichkeit
1/100
Issue-Typ
Bug
Klarheit
Größtenteils klar
Aktivitätsstatus
Veraltet
Bereich
backend

Rechercherichtung

Start by comparing Data.List.NonEmpty.intersect and intersectBy with the corresponding NonEmptyArray functions, as the issue suggests. Review the discussion in #184 and the linked pull request #217 before deciding what outcome is intended. Done means the intersection functions handle empty results consistently with the chosen API direction.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

good first issue type: bug

I think they should be changed to match the implementation of the corresponding functions in NonEmptyArray, i.e. return a List instead of a NonEmptyList.

Alternatively, I saw the discussion in #184 about potentially removing the set functions from Array, List, NonEmptyList, NonEmptyArray, etc. and replacing them with an ordered set type. This might be more evidence supporting those who want to remove them.

Vorherrschende Sprache
PureScript
Sterne
61
Forks
50
Ø Merge
2 T. 6 Std.
Gemergte PRs (30 T.)
1

Entwicklungsumgebung

  • Kein Dockerfile und keine Docker-Compose-Datei
  • Hat eine Pull-Request-Vorlage
  • Kein Beitragsleitfaden

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus purescript/purescript-lists

Alle Issues in purescript/purescript-lists

Ähnliche Issues

Weitere Issues zu Backend & API Design

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.