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

add render_content method to Nm::Context #22

Merged
merged 1 commit into from
Jun 22, 2021
Merged

add render_content method to Nm::Context #22

merged 1 commit into from
Jun 22, 2021

Conversation

kellyredding
Copy link
Member

This method is used to render raw template content strings and is
composed by the render method that takes a template path. This
keeps things DRY while adding a more flexible API.

Specifically, this is needed to implement render Nm templates in
Rails' template engine.

I also added some equality methods to ease testing in the rails-nm
gem that I'm working on.

This method is used to render raw template content strings and is
composed by the `render` method that takes a template path. This
keeps things DRY while adding a more flexible API.

Specifically, this is needed to implement render Nm templates in
Rails' template engine.

I also added some equality methods to ease testing in the rails-nm
gem that I'm working on.
Copy link
Member

@jcredding jcredding left a comment

Choose a reason for hiding this comment

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

Cool by me!

@kellyredding kellyredding merged commit 4cd0ec4 into main Jun 22, 2021
@kellyredding kellyredding deleted the kr/tweaks branch June 22, 2021 20:37
kellyredding added a commit that referenced this pull request Jun 23, 2021
* 226911d - (hotfix) update to the latest much-style-guide
* add render_content method to Nm::Context #22
* rework Nm::Template as Nm::TemplateBehaviors mixin #21
* prep for reworking template rendering to support e.g. Rails rendering #20
  * 066ad42 - add Nm::Source `file_path!` method
  * 6965995 - rework Source to use keyword arguments; tweak argument handling
  * 2c05771 - remove `render` Template methods
* update to our latest gem conventions #19
  * 611e24e - configure t.rb and l.rb; lint the source
  * 40da810 - update to the latest Assert
  * 44bd8a3 - update to our latest gem conventions
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