regexp.js icon indicating copy to clipboard operation
regexp.js copied to clipboard

Release the parser as a separate npm package

Open mathiasbynens opened this issue 10 years ago • 10 comments

  • [x] release the parser as a separate npm package: done https://github.com/jviereck/regjsparser
  • [ ] use it as a dependency for regexp.js

mathiasbynens avatar Nov 03 '13 07:11 mathiasbynens

See https://npmjs.org/package/regexp.js-parser.

I've added you to the list of collaborators at

https://github.com/jviereck/regexp.js-parser

Also setup Travis testing and added a few more RegExp to test the parser against (took them from the test262 suite).

Mathias, can you take a quick look if this looks good and if this is what you are looking for? Happy to change things around and make your life easier :)

For now, I don't want to make regexp.js-parser a dendency of RegExp.JS. Once I got feedback from you, that the basic repo shape is "okay", I will change things around.

jviereck avatar Nov 03 '13 16:11 jviereck

It sure looks alright to me :) Thanks for doing this!

mathiasbynens avatar Nov 03 '13 17:11 mathiasbynens

How about a simpler project name? Something like regexparse would be a bit easier to type/remember. (It would also help when using the binary, as it inherits the npm project name.)

mathiasbynens avatar Nov 03 '13 18:11 mathiasbynens

Agreed that the project name is too complicated.

To keep the "flavor" of RegExp.JS and strengthen the JS part, how about regjsparse?

jviereck avatar Nov 03 '13 19:11 jviereck

I prefer regexparse as it’s more obvious, but sure :) Anything that gets rid of the . and - in the name is a win as far as I’m concerned!

mathiasbynens avatar Nov 03 '13 19:11 mathiasbynens

Renamed the project.

  RegExp.JS - Parser is dead - long live RegJSParser!

https://github.com/jviereck/regjsparser

jviereck avatar Nov 03 '13 19:11 jviereck

Yay! Let’s publish it on npm under this name too, before people start relying on the old name.

mathiasbynens avatar Nov 03 '13 20:11 mathiasbynens

Yay! Let’s publish it on npm under this name too, before people start relying on the old name.

I unpublished the old one and published the new one. npmjs.org seems down fro me, so I cannot verfiy at the moment.

jviereck avatar Nov 03 '13 20:11 jviereck

Any reason you're not using the separate project as a dependency yet?

jamestalmage avatar Jul 31 '15 20:07 jamestalmage

Any reason you're not using the separate project as a dependency yet?

Mostly lack of motivation and time. Using the separate parser project won't be trivial as the parser output has changed compared to what is used in the tree here.

jviereck avatar Aug 05 '15 12:08 jviereck