marijnh / Eloquent-JavaScript

The sources for the Eloquent JavaScript book
https://eloquentjavascript.net
2.99k stars 790 forks source link

fix: add missing comma to range #558

Closed dylancom closed 2 years ago

dylancom commented 2 years ago

The current solution checks for an exact match of 7 characters, but It should be 7 or more characters.

marijnh commented 2 years ago

It works, though (a regexp matches a string if it matches any part of that string), so in the spirit of that exercise I don't think adding an extra character is a good idea.