Skip to content
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

Adding support for passing previous template resolutions into variable templates #70

Merged
merged 3 commits into from
Feb 19, 2018

Conversation

luke-rogers
Copy link
Collaborator

When a template is resolved we add it to the context. The first time a template is resolved the it is added to the resolutions just as a value. However if that same template is resolved again then we convert the value into an array of values for that template. This should make it a bit easier to work with if your only using a template once in a string, but still supports using it multiple times.

@luke-rogers luke-rogers self-assigned this Feb 18, 2018
@codecov
Copy link

codecov bot commented Feb 18, 2018

Codecov Report

Merging #70 into master will increase coverage by 0.1%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master      #70     +/-   ##
=========================================
+ Coverage   97.52%   97.62%   +0.1%     
=========================================
  Files           8        8             
  Lines         242      253     +11     
  Branches       71       74      +3     
=========================================
+ Hits          236      247     +11     
  Misses          6        6
Impacted Files Coverage Δ
lib/splainContext.js 100% <100%> (ø) ⬆️
lib/splainToken.js 96.96% <100%> (ø) ⬆️
lib/splain.js 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8cf1f48...41d5d1a. Read the comment docs.

@mog13 mog13 merged commit 06a4261 into master Feb 19, 2018
@mog13 mog13 deleted the variable-templates-context branch April 17, 2018 21:33
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.

2 participants