Skip to content
This repository has been archived by the owner on Jul 6, 2023. It is now read-only.

Integrate with Temple #147

Open
frm opened this issue Mar 22, 2021 · 1 comment · Fixed by #152
Open

Integrate with Temple #147

frm opened this issue Mar 22, 2021 · 1 comment · Fixed by #152
Labels
enhancement New feature or request

Comments

@frm
Copy link
Member

frm commented Mar 22, 2021

Temple is a fine DSL for our purposes. We're not currently supporting it but it make be a fine addition to our repertoire.

@frm frm changed the title Integrate with Template Integrate with Temple Mar 22, 2021
@frm frm added the enhancement New feature or request label Mar 22, 2021
gabrielpoca added a commit that referenced this issue Apr 30, 2021
In this change:

* Adds support for Temple.
* Updates specs to ignore some Dialyzer warnings.

There's no documentation regarding temple support because I still need to do some more thorough testing. Also, I'm not particularly eager to use FrontMatter with Temple, but I can't think of another way.

Closes #147
@gabrielpoca gabrielpoca reopened this May 12, 2021
@gabrielpoca
Copy link
Member

I'm re-opening this issue because I don't like the path I was taking with this integration. Temple is a DSL, and it doesn't make sense to treat it as just another templating engine that outputs HTML. Temple could be a more dynamic preprocessor or something more than a preprocessor. For instance, I don't think it makes sense to mix Frontmatter with Temple. Instead, I would like to add a DSL to change metadata in Temple. But because we use Temple as a "function" that takes a string and outputs a string, we can't make that.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants