marco-gagliardi / presencejs

A JavaScript library that uses HTML5 controllers to detect whether someone is in front of the computer.
MIT License
18 stars 1 forks source link

Various improvements #2

Closed m00s closed 9 years ago

m00s commented 9 years ago

Ciao Marco, una cosa tira l'altra e ho finito per ristrutturare un po' tutto il progetto. Faccio una pull request unica, poi a te valutare se la strada piace, altrimenti terrò un fork mio per lo sviluppo.

Elenco le modifiche:

Il conflitto in questa pull request è sul README, che hai aggiornato quando ero in sviluppo. Ho comunque aggiunto anch'io l'url alla tua demo, quindi nulla andrebbe perso.

Manca la licenza, che andrebbe messa quanto prima. In bower e npm ho messo MIT, si può cambiare prima di registrare i pacchetti se ne preferisci altre :wink:

Se hai dubbi domande o considerazioni sul codice sarò lieto di risponderti.

m00s commented 9 years ago

Ah dimenticavo, come vedi ci sono due commit di versione, perchè ho bumpato bower e npm separatamente dimenticato di fare un rebase prima della pull request. Andrebbero uniti, alla prossima me ne ricorderò

marco-gagliardi commented 9 years ago

Dire che sei stato utilissimo è dire poco! Procederò al merge e visto l'impegno, se sei d'accordo, vorrei che apparissi anche tu tra gli autori