gr2m / initials

extract initials from and add initials to names
https://gr2m.github.io/initials
MIT License
44 stars 14 forks source link

Auto deploy #17

Closed zoepage closed 9 years ago

gr2m commented 9 years ago

this does not yet fully finish #13, initials.js is not commonJS and there is no build task, but we can do this in two steps. But then do not move the index.html to demo/index.html, because it will break gh-pages. The index.html will be accessible at http://gr2m.github.io/initials/demo instead of http://gr2m.github.io/initials

gr2m commented 9 years ago

The way you export initials in https://github.com/gr2m/initials/blob/auto-deploy/index.js will make this fail

var initials = require('initials')
initials('John Doe')

Instead we'd need to do

var initials = require('initials').initials
initials('John Doe')

Which is not what we want. I would simply do

var exports = module.exports = require('./lib/initials')

Later we can then split up the addTo, find and parse functions into multiple files, but let's do it in another PR :)

gr2m commented 9 years ago

Looking great, last comments, then I'll merge it and if necessary create follow up issues. Good work

gr2m commented 9 years ago

Looking great, last comments, then I'll merge it and if necessary create follow up issues. Good work

gr2m commented 9 years ago

merged via 932c043

gr2m commented 9 years ago

I've cleaned up your git commits, usually it should show that they have originally authored by you, and only commited by me, I'm sorry that your authorship was lost, I really didn't mean to take over your contributions :( I'll let you clean up commits in future to make sure that this won't happen again, I'm really sorry