exercism / exercism/problem-specifications

markdown: add test for multiple lists

Aperta
#1,664 8 commenti 1 reazione 0 assegnatari Vedi su GitHub
hold
Lingua principale
Ruby
Stelle
358
Fork
563
Merge medio
18h 41m
PR unite (30g)
2

Descrizione

I was just going to suggest adding an additional test to ensure that the code handles multiple lists properly. In my solution to the problem, I used a regex to insert the `

    ` tags at the appropriate locations. Originally I had just `(
  • .*
  • )` which passed all the tests, but which will work ONLY WHEN YOU HAVE A SINGLE LIST in your markdown text. If you have more than one, it'll put a single pair of `
      ` tags around all of them (beginning of the first, end of the last) and whatever headers and paragraphs are found between.

      My current regex `(

    • .*?
    • )(<[ph])|(
    • .*
    • )` handles multiple lists correctly, as far as I can tell. The first half (before the '|') catches string-initial or internal lists, while the latter half will kick in to handle string-terminal lists.

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.