Add support for character classes `[...]`
Fixes #249
It also makes the quantifiers more expressive:
I.e., now it supports: {,4}, {4}, {1,3}, {1,} instead of just {1,3} and {1,}
@StefanosChaliasos thanks for this contribution! A bit busy today but I'll try to review this at some point in the evening.
coverage: 98.008% (-1.6%) from 99.613% when pulling df53edcc873dadb2586facfa65cba75dbf0eb063 on StefanosChaliasos:add-support-for-charclass into 1bdf9b7c94f3681976b41165044c626a768fb560 on caleb531:develop.
Will go over everything tomorrow. Thanks a lot for the feedback.
I did some more changes, can you review the new ones. Basically I added support for shorthand (e.g., '\d') and I tokenised whitespace. I need to add more tests and polish the code. I'll change the PR as a draft until done.
Hey, @StefanosChaliasos! I left some additional comments on the PR—apologies if they seem nitpicky, but just wanting to maintain solid code quality and consistency for this project.
Thanks for the review, I will address the comments once I find some time
Thanks for the additional comments, will fix everything by the end of the week. Got busy with other stuff :)
@StefanosChaliasos just want to check in briefly since this will now require a rebase on the develop branch because of some changes that have been made on our end. The comments I made earlier are relatively minor, so let me know if you'd like any help landing this. I think we want to cut a new release before too long so it would be awesome to have these changes included 👍🏽