Comments (17)
Shouldn't be difficult, if you want to do a pull request I'd be happy to merge it in. Or give me a couple days and I'll come up with something. thanks
from handlebars-helpers.
I don't use coffeescript - if my pull script updates the lib/helpers/helpers-collection.js and the unit test, would that be ok?
from handlebars-helpers.
Absolutely. I'm easy going, any contribution is cool with me, I'd be happy to convert to coffee for you (we might get rid of it anyway, I'm not a big fan of having to code the helpers in coffee)
from handlebars-helpers.
Pull request sent. Comments in there for the details and if you don't want it as I did it, I understand. Thank you for this project - I love Handlebars and these libraries are awesome.
from handlebars-helpers.
👍 thanks for the pull request! I'll get it pulled in sometime today.
from handlebars-helpers.
btw, you might be interested in the experimental helpers I maintain here: https://github.com/assemble/handlebars-helpers-examples/tree/master/experimental/helpers
Some of them aren't even finished... but I keep them pushed up just in case they help inspire other ideas.
from handlebars-helpers.
Btw - I figured out to use grunt to run the tests locally and I've gotten a coffeescript version of eachIndexPlusOne along with the test done too. It is "more" proper, so I won't be offended if you nuke my pull request. I can definitely make a new one.
from handlebars-helpers.
did you make the coffee changes to the same clone? if so you can just push it up again and it will add to this request automatically. Or we can nuke this and you can just do a new request if that's not too much trouble. Either way, let me know what works for you. thanks!
from handlebars-helpers.
btw, do you have any clue as to why there was so much activity on this repo today? don't get me wrong it's pretty awesome for sure lol, but we got a lot of stars today and I'm just curious as to why
from handlebars-helpers.
I'll be honest - I'm a Git neophyte. I use the hell out of it for my
projects, but cloning/pull requests still confuse me. I'm worried a new
push of the same stuff would keep both my mods in and obviously the JS mods
need to go away since the Grunt task overwrites em. I'll do whatever makes
it easier to get it in though (hell, even emailing the two files ;)
On Fri, Aug 2, 2013 at 4:42 PM, Jon Schlinkert [email protected]:
did you make the coffee changes to the same clone? if so you can just push
it up again and it will add to this request automatically. Or we can nuke
this and you can just do a new request if that's not too much trouble.
Either way, let me know what works for you. thanks!—
Reply to this email directly or view it on GitHubhttps://github.com//issues/43#issuecomment-22037811
.
Raymond Camden, Adobe Developer Evangelist
Email : [email protected]
Blog : www.raymondcamden.com
Twitter: cfjedimaster
from handlebars-helpers.
I tweeted it. I've got mildly high visibility (5.9k followers), but I'm no
super star.
On Fri, Aug 2, 2013 at 4:43 PM, Jon Schlinkert [email protected]:
btw, do you have any clue as to why there was so much activity on this
repo today? don't get me wrong it's pretty awesome for sure lol, but we got
a lot of stars today and I'm just curious as to why—
Reply to this email directly or view it on GitHubhttps://github.com//issues/43#issuecomment-22037885
.
Raymond Camden, Adobe Developer Evangelist
Email : [email protected]
Blog : www.raymondcamden.com
Twitter: cfjedimaster
from handlebars-helpers.
well that might be why lol. 5.9k is nothing to sneeze at. thanks a lot!
from handlebars-helpers.
I'm worried a new push...
Well, if it's all the same to you, I'd hate to pass up this chance for you to see what happens when you do it lol. Since your code won't do anything to effect this repo unless the pull request is merged in, why don't you just push it up to see what happens. And then we can delete/close this and you can do a new one from scratch if it's messed up. It's impossible to offend the Git Gods more than I have... I promise you lol.
from handlebars-helpers.
Ok - here goes nothing then...
On Fri, Aug 2, 2013 at 4:51 PM, Jon Schlinkert [email protected]:
I'm worried a new push...
Well, if it's all the same to you, I'd hate to pass up this chance for you
to see what happens when you do it lol. Since your code won't do anything
to effect this repo unless the pull request is merged in, why don't you
just push it up to see what happens. And then we can delete/close this and
you can do a new one from scratch if it's messed up. It's impossible to
offend the Git Gods more than I have... I promise you lol.—
Reply to this email directly or view it on GitHubhttps://github.com//issues/43#issuecomment-22038342
.
Raymond Camden, Adobe Developer Evangelist
Email : [email protected]
Blog : www.raymondcamden.com
Twitter: cfjedimaster
from handlebars-helpers.
Ok, the Pull Request contains both commits. My first commit is definitely bad. I missed the module export in the JS. Of course, it will also be blown away when you run grunt build. So my opinion is this. Accept the pull request (again, assuming you are cool with the name), run a grunt build locally, not that the collection JS file will be overwritten - and then commit that yourself to make it 'cleaner' (imo) since people (like me) will skip the coffeescript versions probably and just use the JS. Hope that makes sense. I need beer now.
from handlebars-helpers.
sounds great, the code looks great, and I'll do what you suggested.
I need beer now.
ha, have a good one
thanks!
from handlebars-helpers.
Does eachIndex+1
was implemented?
from handlebars-helpers.
Related Issues (20)
- Question - using pre-made helpers, can you split on two characters? HOT 3
- Can you use both split and unique, in that order? HOT 2
- Comparation(#compare var1 "==" var2) with two variables doesn't work HOT 1
- Upgrade for new versions of the related plugins HOT 1
- does not export a 'register' function, cannot import HOT 1
- Create SECURITY.md
- Not splitting string by line breaks '\n' HOT 1
- todo: merge updates from https://github.com/Budibase/handlebars-helpers HOT 3
- Markdown Helper is broken...
- Vulnerability found (due to highlight.js) HOT 3
- isLast in nested forEach
- Titleize handlebar doesn't titleize input string correctly.
- Request: encode to hex HOT 1
- If loading all helpers the String helper reverse overrides the Array helper reverse. HOT 2
- An examples of working out averages on columns
- Handle empty fields
- String helper `split` doesn't split by empty string.
- This package does not install
- Fumanchu - A supported Handlebar + Handlebars-Helpers together! HOT 4
- query online
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from handlebars-helpers.