krakenjs / swaggerize-express

Design-driven apis with swagger 2.0 and express.
Other
354 stars 81 forks source link

update regex in "bleb" unit test. #116

Closed tongfa closed 7 years ago

tongfa commented 7 years ago

I believe this is a safe modification because the regexes are equivalent. The difference amounts to adding (?: ... ) which inserts a non matching group. I verified (by hand) that in chrome v8 inserting a non matching group inside a matching group does not affect the matching group. The outer group still matches.

subeeshcbabu-zz commented 7 years ago

👍