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

Jade is depricated #83

Closed
MrKieren opened this issue Apr 21, 2016 · 15 comments
Closed

Jade is depricated #83

MrKieren opened this issue Apr 21, 2016 · 15 comments

Comments

@MrKieren
Copy link

grunt-html2js has a dependency for jade ^1.3.1 which now deprecated:

npm WARN deprecated [email protected]: Jade has been renamed to pug, please install the latest version of pug instead of jade

@idurotola
Copy link

has this been fixed or does it auto detect .pug files too?

@longrunningprocess
Copy link

@MrKieren ,you might consider a PR for this one just in case @karlgoldstein isn't able to get to this anytime soon.

@agenaille
Copy link

+1 We have upgraded to Pug and have just found our templates are no longer rendering to JS, as we renamed all the .jade templates to .pug. I guess this'll take a major release to fix. Any support would be much appreciated! Very nice module by the way!

bennorwood pushed a commit to bennorwood/grunt-html2js that referenced this issue Nov 9, 2016
…ease package.json update reflects the addition). Alphabetized dependency list and added pugg as an optional dependency.
bennorwood added a commit to bennorwood/grunt-html2js that referenced this issue Nov 9, 2016
rquadling#83 Add pug rendering capabilities and unit tests (major release pack…
bennorwood pushed a commit to bennorwood/grunt-html2js that referenced this issue Nov 9, 2016
bennorwood added a commit to bennorwood/grunt-html2js that referenced this issue Nov 9, 2016
tyriker pushed a commit to WeWriteCode/grunt-html2js that referenced this issue Jan 17, 2017
…ease package.json update reflects the addition). Alphabetized dependency list and added pugg as an optional dependency.
tyriker pushed a commit to WeWriteCode/grunt-html2js that referenced this issue Jan 17, 2017
@tyriker tyriker mentioned this issue Jan 18, 2017
@Abdisalan
Copy link

Is this issue due been fixed soon? I see an open PR from Jan 17

rquadling added a commit that referenced this issue May 19, 2017
#83 Added pug template rendering capabilities.
@longrunningprocess
Copy link

@rquadling , mind if we get a 0.4.0 tag on this?

@rquadling
Copy link
Owner

Oh! Sorry!!!

@longrunningprocess
Copy link

no problem, my workflow for tags almost always involves someone reminding me to tag it ;-) appreciate the quick turnaround

@longrunningprocess
Copy link

Looks like this isn't fixed yet so the same deprecation warnings will continue until this is changed: https://github.com/rquadling/grunt-html2js/blob/master/package.json#L48

@Abdisalan @rquadling FYI

@rquadling
Copy link
Owner

One issue I cannot resolve is if someone is using Jade templates, Pug will not pick up on Jade includes.

So, for the time being, V0.4 will support both pug and jade (with the deprecation notice). V0.5 will deprecate jade support entirely.

@longrunningprocess
Copy link

fair enough, sorry it's been such a nuisance. I can definitely live with the warnings.

@rquadling
Copy link
Owner

Not a nuisance. I'm new to JS at this level. I'm a PHP dev, but some of the tools I use have been unsupported for such a long time that taking them over and maintaining them (to the best of my ability) was the only option.

OOI, I've not used Jade or Pug and only rely on the unit tests to say things are OK. How popular are they?

I'll release a 0.5.0 next week so that allows anyone still needing Jade to have a forward compatible version (0.4.0) to migrate their templates with.

@longrunningprocess
Copy link

That's a tough position to be in, sorry that fell on you man!

I have no idea how popular those are, I'd never heard of them until these deprecations started popping up.

Thanks for the upcoming release, hang in there!

@agenaille
Copy link

@rquadling In regards to supporting jade and pug in v0.4 and then deprecating jade in v0.5, wouldn't that potentially break dev teams html2js builds whom are still using jade? Essentially anyone using a package.json dependency value similar to this would automatically pull in v0.5 and break their jade templates html2js build:

"grunt-html2js": "^0.3.5",

Sticking to semantic versioning, since a release like this could possibly break backwards compatibility, it may be wise to do a 1.0.0 release so that developers taking all patch and minor releases are not affected. What do you think?

@rquadling
Copy link
Owner

In semver, V0 is "special". http://semver.org/#spec-item-4 / https://getcomposer.org/doc/articles/versions.md#caret-version-range-

So ^0.4 will allow for all release < 0.5.0

So, if there is anything that needs fixing and is specifically Jade related, then it can happily sit in the 0.4 branch.

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 a pull request may close this issue.

6 participants