-
Notifications
You must be signed in to change notification settings - Fork 33
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Reworking the syntax checker #66
Comments
This landed in the master branch. I will manage to get some time for a beta release, hopefully next week! |
please also describe the upgrade guide (our current ruleset file references classes which are removed by your refactoring). |
Here's a quick update: I'm still working on this! I have had less time lately (became CTO at my workplace) but kept working on the rewrite. The new engine is completed and fixes all the problems listed above and also prepares for the future (like arrow functions support). It sleeps in the master, waiting to be tagged for beta. I just added a branch with a quick method for configuring the engine for simpler needs that should be merged in september. Next steps are:
Thank you for using Twigcs, and thank you for your patience! |
All is done, we should be ready for 4.0-beta soon! |
Really nice work 👍 |
I'm currently reworking the way Twigcs handles basic style checks like counting the number of spaces, checking that something is lowercased etc... It will be a separated engine that can works along the unused macros/variable checkers. At the moment it is called "RegEngine" because it heavily uses regular expression.
The branch: https://github.com/OwlyCode/twigcs/commits/experimental
I expect this rework to fix the following issues :
The text was updated successfully, but these errors were encountered: