Skip to content

Add chained else expression support#629

Merged
sunng87 merged 2 commits into
sunng87:masterfrom
progmboy:else_chain
Jan 16, 2024
Merged

Add chained else expression support#629
sunng87 merged 2 commits into
sunng87:masterfrom
progmboy:else_chain

Conversation

@progmboy

Copy link
Copy Markdown
Contributor

add support for #12 and #586

@progmboy progmboy changed the title add chained else expression support Add chained else expression support Jan 16, 2024
@coveralls

coveralls commented Jan 16, 2024

Copy link
Copy Markdown

Coverage Status

coverage: 81.605% (+0.3%) from 81.329%
when pulling 8f16353 on progmboy:else_chain
into 23672e8 on sunng87:master.

Comment thread src/render.rs
render_helper(self, registry, ctx, rc, out)
}
}

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If I understand correctly this implementation can be removed.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, you're right. I forgot to delete this implementation.

@sunng87 sunng87 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you so much @progmboy ! The implementation is brilliant. Please check my comment about the Renderable implement.

@sunng87

sunng87 commented Jan 16, 2024

Copy link
Copy Markdown
Owner

Let me merge this first and resolve minor issues in another patch.

@sunng87 sunng87 merged commit e30d8ab into sunng87:master Jan 16, 2024
This was referenced Jan 16, 2024
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