Skip to content

Deferred engine load #616

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

Merged
merged 6 commits into from
Feb 8, 2017
Merged

Conversation

bmuenzenmeyer
Copy link
Member

Addresses #610

Summary of changes:

  • Allows the caller to determine when to load all engines, removing this check from the module scope of pattern_engine.js. This should help control the experience working with core a bit more.
  • This also includes a merge for #608 Change var to const or let which encompassed a bunch of stellar work by @benhalverson

There is still one broken unit test, but I intend to look into that separately.

part of #610
part of the merge of #608

most unit tests working, only a few left - due to me hanging a property off the engine that @geoffp was not doing.
still need to see how it behaves during actual builds
still reporitng quite a few handlebar tests broken
still reporitng quite a few handlebar tests broken

part of #610
…sequently falling through to mustache

only one failing unit test now

part of #610
@bmuenzenmeyer
Copy link
Member Author

One thing I am not sure of is why I don't see the commits from @benhalverson from #608. I had to merge that manually which led to this work, but I had expected them to show up here. Any ideas why they aren't reflected? I want you to get credit for those commits. Does this bother you at all Ben?

@bmuenzenmeyer bmuenzenmeyer merged commit d5fbacf into dev-3.0 Feb 8, 2017
@bmuenzenmeyer
Copy link
Member Author

@raphaelokon can you retry your CLI against dev-3.0 ?

@raphaelokon
Copy link
Contributor

@bmuenzenmeyer Sure will do. Thanks.

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.

3 participants