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

Aberta
  • Algo errado?
#229 3 comentários 0 reações 0 responsáveis Ver no GitHub

Ninguém assumiu esta issue ainda.

  • #230 de @JordanMartinez — fechado sem integrar

Avaliação

Dificuldade
2/5
Tempo estimado
1-3 horas
Facilidade para iniciantes
25/100
Tipo de issue
Bug
Clareza
Razoavelmente clara
Status de atividade
Estagnada
Domínio
backend

Direção de pesquisa

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.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

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.

Linguagem predominante
PureScript
Estrelas
58
Forks
70
Merge médio
2d 22h
PRs com merge (30d)
1

Preparar o ambiente

  • Sem Dockerfile nem arquivo Docker Compose
  • Tem um modelo de pull request
  • Sem guia de contribuição

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de purescript/purescript-arrays

Todas as issues de purescript/purescript-arrays

Issues semelhantes

Mais issues de Backend & API Design

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.