Skip to content

Conversation

@MaddieM4
Copy link

This work is IN PROGRESS, and probably not adequate for a merge yet! However, it does work quite well for the toy examples I've been working on so far.

TODO:

  • Various syntax cleanup.
  • Consider including a convenience function or script to rebuild a tree of separate AMD modules.
  • Get maintainer feedback. There are places where this code is logically correct, but doesn't "do as the Romans do", for example some changes to quote literals, and using literals at all where the existing code goes through multiple layers of abstraction all the way down to templates.

While I was frustrated by the nondeterministic build process, and worried how that would make the pull request unreadable, Github luckily does not show the large and unnecessary diffs for those files.

@MaddieM4
Copy link
Author

Used the Minilla branch to enable me to do local testing, developed a patch to fix the broken tests (really, the broken behavior in compile_template_content), and applied the patch to this Minilla-free PR branch. Now it passes Travis! Life is good.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant