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

Why not totally replace Swig with Nunjucks? #1593

Open
mamboer opened this Issue Nov 17, 2015 · 8 comments

Comments

8 participants
@mamboer
Copy link

mamboer commented Nov 17, 2015

I notice that the code base of Hexo uses Swig along with Nunjucks.
Swig isn't maintained currently, why not just replace all the Swig stuff with Nunjucks

@PhiLhoSoft

This comment has been minimized.

Copy link

PhiLhoSoft commented Nov 20, 2015

Swig, Nunjucks, EJS, that's lot of template engines for one application...
Alas, if we use an existing template, EJS is probably mandatory, as all templates I looked at use it.
Unless there is an EJS to Nunjucks converter somewhere?

@Xuanwo Xuanwo added the question label Dec 14, 2015

@ivanoats

This comment has been minimized.

Copy link

ivanoats commented Jan 31, 2016

I'd agree with @mamboer on this one. It's a bit scary to look at the giant "unmaintained" notice on Swig's github repo. Nunjucks is the successor to Swig. This question should be answered because the current Hexo Docs say that Swig is the default tempting language. https://hexo.io/docs/themes.html#layout

An alternative to Nunjucks may be liquid-node which may be better because Liquid is the default templating language for Jekyll.

@leesei

This comment has been minimized.

Copy link
Member

leesei commented Feb 23, 2016

I see a recent commit 35342cb to replace swig in creating post.
But swig renderer is still registered in the Hexo core, this is against to movement to modularize Hexo.
The default theme Landscape uses ejs, our setup teaches users to install external renderer. I think swig has served it's purpose and its time to say goodbye.

@tommy351, maybe we should announce swig renderer's deprecation and remove it altogether in the next release.

@Xuanwo

This comment has been minimized.

Copy link
Contributor

Xuanwo commented Feb 23, 2016

Agree with @leesei too.

@h404bi

This comment has been minimized.

Copy link
Contributor

h404bi commented Nov 1, 2016

Swig is dead. paularmstrong/swig@70a1c88

@NoahDragon

This comment has been minimized.

Copy link
Member

NoahDragon commented Nov 2, 2016

Agree.

@dumindu

This comment has been minimized.

Copy link

dumindu commented Feb 25, 2018

Any update on this?

@NoahDragon

This comment has been minimized.

Copy link
Member

NoahDragon commented Mar 15, 2018

@dumindu please check the status of PR #2903

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment