-
Notifications
You must be signed in to change notification settings - Fork 416
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
Refactor compiler for clarity #2091
Conversation
Co-authored-by: Blake Williams <blakewilliams@github.com>
1 file(s) had their final line ending fixed:
|
c968b8b
to
ccac374
Compare
1 file(s) had their final line ending fixed:
|
@BlakeWilliams thanks for the thorough review! It was fun to look at ways of incorporating your feedback. I can't help but feel like there is a whole 'nother pass to be done on the compiler. We have a lot of similar logic around finding conflicting render paths that remains awkward as a whole. I wonder if we should land this PR in more or less its current shape and then do a second pass in a follow up? |
Per @BlakeWilliams's suggestion, I've refactored the compiler to make it easier to read and understand. I'm sure we could take this even further, but I stopped once I had a clean
Template
class extracted. Any and all feedback welcome ❤️