`groupBy` has unexpected `l` argument for `a -> a -> Boolean` predicate

Ouverte
#229 3 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

  • #230 par @JordanMartinez — fermée sans fusion

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
25/100
Type d'issue
Bug
Clarté
Plutôt claire
Activité
À l'abandon
Domaine
backend

Piste de recherche

Start with the linked try.purescript.org example and inspect the groupBy implementation and its tests to trace how the predicate arguments are chosen. The report expects adjacent pairs (0, 1) and (1, 2) for [0, 1, 2]; determine whether the behavior is unintended, then add a regression test and confirm it passes if a change is warranted.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

See https://try.purescript.org/?gist=26968458c1261226b33e574128ba9aae

For a given array like [0, 1, 2], I expect the l and r arguments to be

  • first run: 0 and 1
  • second run: 1 and 2

However, in the second run, l is 0 rather than 1.

I'd argue that the behavior here is unexpected.

Langage dominant
PureScript
Étoiles
58
Forks
70
Merge moyen
2 j 22 h
PR mergées (30 j)
1

Préparer son environnement

  • Aucun Dockerfile ni fichier Docker Compose
  • Propose un modèle de pull request
  • Aucun guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de purescript/purescript-arrays

Toutes les issues de purescript/purescript-arrays

Issues similaires

Plus d'issues Backend & API Design

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.