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

fix: merger substring exception #135

Merged
merged 3 commits into from
Jul 25, 2022
Merged

fix: merger substring exception #135

merged 3 commits into from
Jul 25, 2022

Conversation

erdos
Copy link
Owner

@erdos erdos commented Jul 23, 2022

There is a bug in the merger algorithm causing some templates to crash stencil.

It is a miracle it has not came up earlier.

@erdos erdos added the syntax template language syntax label Jul 23, 2022
@codecov
Copy link

codecov bot commented Jul 23, 2022

Codecov Report

Merging #135 (7726350) into master (24621ee) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #135   +/-   ##
=======================================
  Coverage   93.12%   93.12%           
=======================================
  Files          25       25           
  Lines        1906     1906           
  Branches      102      102           
=======================================
  Hits         1775     1775           
  Misses         29       29           
  Partials      102      102           
Impacted Files Coverage Δ
src/stencil/merger.clj 99.08% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us.

@erdos erdos merged commit ffe65f3 into master Jul 25, 2022
@erdos erdos deleted the fix-merger-substring branch July 25, 2022 07:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
syntax template language syntax
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant