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

Pattern param rework #771

Merged
merged 19 commits into from
Jan 15, 2018
Merged

Pattern param rework #771

merged 19 commits into from
Jan 15, 2018

Conversation

bmuenzenmeyer
Copy link
Member

Addresses #769 Rework Pattern Parameter Algorithm

Summary of changes:

  • Found pattern parameters are no longer rendered immediately. Not sure why I ever thought that was a good idea. Now, they are evaluated and replace their value or section in the pattern's extendedTemplate
  • All tests are passing - excepting two which I commented out.
  • I expect that list_item_hunter should be similarly refactored.

@bmuenzenmeyer bmuenzenmeyer self-assigned this Jan 15, 2018
@coveralls
Copy link

coveralls commented Jan 15, 2018

Coverage Status

Coverage increased (+7.4%) to 71.349% when pulling 2f84d9c on pattern-param-rework into 578a9af on dev.

@bmuenzenmeyer bmuenzenmeyer merged commit 63117e2 into dev Jan 15, 2018
@bmuenzenmeyer bmuenzenmeyer deleted the pattern-param-rework branch January 15, 2018 18:20
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