Skip to content
This repository has been archived by the owner on Apr 11, 2018. It is now read-only.

delete useless code #164

Merged
merged 1 commit into from Dec 11, 2012
Merged

delete useless code #164

merged 1 commit into from Dec 11, 2012

Conversation

lepture
Copy link
Contributor

@lepture lepture commented Dec 11, 2012

this code is just useless repeating.

@lepture
Copy link
Contributor Author

lepture commented Dec 11, 2012

Hi, I am maintain another branch at https://github.com/lepture/swig/tree/jinja now, and I have published it to npm as jinja for my own usage. Just for your knowledge.

@paularmstrong
Copy link
Owner

@lepture Re-publishing as jinja is not a very good idea. Swig does not have the same api as Jinja, so reusing the name is very misleading. There's also already a jinja.js as well. It also may discourage active development and interest in Swig itself.

If you're frustrated with the speed of releases of swig and need to use the newest versions in your deployment, I highly encourage you to use a git url as the dependency version or a git submodule

Example package.json:

{
  "name": "mystuff",
  "dependencies": {
    "swig": "git://github.com/lepture/swig.git#jinja"
  }
}

Maintaining this library is not my full time job. I have a lot of other obligations that take priority over things like Swig, which I do for fun. Please understand and be respectful of my time.

paularmstrong added a commit that referenced this pull request Dec 11, 2012
@paularmstrong paularmstrong merged commit 808286b into paularmstrong:master Dec 11, 2012
@lepture
Copy link
Contributor Author

lepture commented Dec 11, 2012

Hi @paularmstrong I do respect your work, but I don't like the git way, actually I will delete this package in npm when swig is ready for use.

I have taken a note in https://github.com/lepture/nico/blob/master/TODO.md

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants