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

automatically generate readme from source #36

Closed
tkellen opened this issue Nov 12, 2015 · 13 comments · Fixed by #86
Closed

automatically generate readme from source #36

tkellen opened this issue Nov 12, 2015 · 13 comments · Fixed by #86
Assignees

Comments

@tkellen
Copy link

tkellen commented Nov 12, 2015

...using https://www.npmjs.com/package/verb

@phated
Copy link
Member

phated commented Nov 16, 2015

@jonschlinkert - could you help us with this?

@jonschlinkert
Copy link

sure! I'm releasing the next version of verb this week (major overhaul). I'll do this right afterwards

@jonschlinkert
Copy link

jonschlinkert commented Nov 30, 2016

Wow, I just saw this when I closed another issue. I must have completely forgotten about it! If you're still interested in doing this I can take a look (apologies!)

@tkellen
Copy link
Author

tkellen commented Nov 30, 2016

No apologies needed sir! We'd still love help with this if you're willing and interested.

@phated
Copy link
Member

phated commented Apr 18, 2017

Ping @jonschlinkert

@jonschlinkert
Copy link

@phated the reason I haven't done anything with this was that it would be problematic for you given that it requires node v0.4.7.0 or later (b/c of handlebars), and I've been trying to think of a way to fix that problem.

@phated
Copy link
Member

phated commented Apr 18, 2017

@jonschlinkert ah! thanks.

@phated
Copy link
Member

phated commented Apr 18, 2017

@jonschlinkert it seems the lie in their Readme: https://github.com/wycats/handlebars.js#supported-environments 😛

@jonschlinkert
Copy link

Indeed it does, since https://github.com/wycats/handlebars.js/blob/master/package.json#L21 will cause errors in CI's

@phated
Copy link
Member

phated commented Apr 18, 2017

Wow, 0.4.7! They haven't updated that in forever (not to be confused with 4.0.7, haha)

@jonschlinkert
Copy link

jonschlinkert commented Apr 18, 2017

omg does it really say that? lol

holy crap. I'm slightly dyslexic (really) so I do that a lot. Specifically with numbers. Even when I double-check it doesn't look like what you're saying!

edit: btw, this is the third time I can think of that I had a problem reading a dot-delimited number on an issue! same thing, my brain switches the order of the numbers for some reason.

@jonschlinkert
Copy link

I specifically remember getting CI errors related to handlebars that alerted me to the issue. I'll look into it more. if it wasn't the engine version it might be something else in the hbs tree.

@doowb
Copy link
Member

doowb commented Apr 18, 2017

lol... I've looked at Handlebars' package.json many times and thought that it said >=4 also.

remember getting CI errors related to handlebars

I thought I saw those too.

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.

4 participants