{"action":"created","issue":{"url":"https://api.github.com/repos/FattiDiCookies/italianPrivacyPolicy/issues/73","labels_url":"https://api.github.com/repos/FattiDiCookies/italianPrivacyPolicy/issues/73/labels{/name}","comments_url":"https://api.github.com/repos/FattiDiCookies/italianPrivacyPolicy/issues/73/comments","events_url":"https://api.github.com/repos/FattiDiCookies/italianPrivacyPolicy/issues/73/events","html_url":"https://github.com/FattiDiCookies/italianPrivacyPolicy/issues/73","id":92022240,"number":73,"title":"organizzazione del repository","user":{"login":"2autunni","id":12629466,"avatar_url":"https://avatars.githubusercontent.com/u/12629466?v=3","gravatar_id":"","url":"https://api.github.com/users/2autunni","html_url":"https://github.com/2autunni","followers_url":"https://api.github.com/users/2autunni/followers","following_url":"https://api.github.com/users/2autunni/following{/other_user}","gists_url":"https://api.github.com/users/2autunni/gists{/gist_id}","starred_url":"https://api.github.com/users/2autunni/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/2autunni/subscriptions","organizations_url":"https://api.github.com/users/2autunni/orgs","repos_url":"https://api.github.com/users/2autunni/repos","events_url":"https://api.github.com/users/2autunni/events{/privacy}","received_events_url":"https://api.github.com/users/2autunni/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/FattiDiCookies/italianPrivacyPolicy/labels/Discussione%20Richiesta","name":"Discussione Richiesta","color":"e11d21"},{"url":"https://api.github.com/repos/FattiDiCookies/italianPrivacyPolicy/labels/Richiesta%20di%20Approvazione","name":"Richiesta di Approvazione","color":"5319e7"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":6,"created_at":"2015-06-30T08:15:57Z","updated_at":"2015-06-30T15:10:59Z","closed_at":null,"body":"Tra le cose da fare urgentemente (vedi #71 di @Gix075  ) ci sono alcune modifiche al repository e al modo di collaborare\r\n\r\n1. Cambiare struttura delle directory del repository (come proposto in e discusso in #33).\r\n\r\nPer questa si è offerto @raynor85\r\n\r\n@Gix075  propone poi di \r\n2. Assegnare permessi di scrittura solo al gruppo degli amministratori  Implementare per i commit solo il meccanismo delle pull-request (al fine di proteggere il lavoro)\r\n\r\nSecondo me il problema è più articolato:\r\nadesso tutti possono effettuare commit sul master, quindi senza controllo ognuno di noi può introdurre codice non funzionante in produzione, questo al di là di chi sia a compiere il commit (gli errori possono capitare a chiunque) o di quale metodo utilizziamo: pull-request o Feature Branches\r\nInoltre il progetto è composto da due parti: quella legale  e il codice.\r\n\r\nGli aggiornamenti a quella legale sono prima discussi nelle issue  e solo una volta approvati sono portati nel repository: sostanzialmente non c'è accesso concorrente ai file, ma è necessaria una validazione rigorosa e non automatizzabile.  \r\nIl codice invece presenta le classiche problematiche di lavorare in maniera distribuita, possiamo lavorare in maniera concorrente ed è anche possibile pensare un domani di eseguire test automatici sul codice\r\n\r\n### Proposta\r\n\r\nAccettiamo sia il meccanismo della fork  e successiva pull-request che Feature Branches (a seconda del tipo di file e della preferenza dello sviluppatore)\r\nOgnuno esegue commit sul repository solo di codice funzionante o di testi legali su cui c'è consenso nelle issue\r\n\r\nNel repository esistono sempre 3 rami:\r\nMaster: il codice stabile a disposizione dell'utente:  per ora dopo il refactoring lo congelerei fino alla prima versione accettabile\r\nRelease: Ramo dove viene rivisto e testato il codice prima di spostarlo sul master come nuova versione stabile.\r\nDev: lo spazio di lavoro, le pull request vengono accettate solo per questo ramo \r\n\r\nOgnuno può aprire nuovi rami per aggiungere features, rispondere a issue, o fixare bug, ma sempre partendo da dev \r\nUnica eccezione i bugfix alla versione stabile che possono partire da release, se dev è troppo dissimile.\r\n\r\nEleggiamo una persona con la responsabilità di testare il codice sul ramo Release e autorizzare il passaggi  a Master\r\n\r\nPassiamo il codice da dev a release quando c'è sufficiente consenso fra gli sviluppatori\r\nTrattiamo le modifiche ai testi legali sempre come le modifiche al codice: aggiornare un testo porta ad aumentare il numero di versione.\r\nQuesto aiuta l'utente finale a capire che qualcosa è cambiato.\r\nChe ne pensate?\r\n\r\n\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/FattiDiCookies/italianPrivacyPolicy/issues/comments/117221625","html_url":"https://github.com/FattiDiCookies/italianPrivacyPolicy/issues/73#issuecomment-117221625","issue_url":"https://api.github.com/repos/FattiDiCookies/italianPrivacyPolicy/issues/73","id":117221625,"user":{"login":"raynor85","id":5817333,"avatar_url":"https://avatars.githubusercontent.com/u/5817333?v=3","gravatar_id":"","url":"https://api.github.com/users/raynor85","html_url":"https://github.com/raynor85","followers_url":"https://api.github.com/users/raynor85/followers","following_url":"https://api.github.com/users/raynor85/following{/other_user}","gists_url":"https://api.github.com/users/raynor85/gists{/gist_id}","starred_url":"https://api.github.com/users/raynor85/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/raynor85/subscriptions","organizations_url":"https://api.github.com/users/raynor85/orgs","repos_url":"https://api.github.com/users/raynor85/repos","events_url":"https://api.github.com/users/raynor85/events{/privacy}","received_events_url":"https://api.github.com/users/raynor85/received_events","type":"User","site_admin":false},"created_at":"2015-06-30T15:10:59Z","updated_at":"2015-06-30T15:10:59Z","body":"Hai ragione sulla node-modules, iniziando da un git fresco mi ero\ndimenticato di toglierla, e poi me la sono portata dietro nei merge. L'ho\ntolta.\nRiguardo i rami non saprei, io tendo a copiare dagli esempi che vedo in\ngiro, come bootstrap, i quali hanno dev e dist nello stesso repo. Poi\nvalutate il fatto che l'utente medio che scarica da git automaticamente\nscarica / clona da master, non si pone nemmeno il problema di un altro\nbranch. Tuttavia quello che dici ha molto senso. Decidete voi.\n\nRuben\n\n2015-06-30 16:48 GMT+02:00 Marcello Cerruti <notifications@github.com>:\n\n> @raynor85 <https://github.com/raynor85> mi sembra ok: domanda la cartella\n> node-modules non andrebbe ignorata mettendola in .gitignore? non è creata\n> da *npm -install* ?\n>\n> @Gix075 <https://github.com/Gix075> rispondendo al tuo commento #73\n> (comment)\n> <https://github.com/FattiDiCookies/italianPrivacyPolicy/issues/73#issuecomment-117201107>\n>\n> Io la vedevo a rami più che a directory.\n> Va bene avere cartelle dev e dist, ma IHMO servono i rami separati, per\n> dare all'utente finale un prodotto stabile sempre se pesca nel ramo master.\n>\n> Es\n> Se un utente vuole compilare da sè il prodotto finito deve usare la dev di\n> master che sarà sempre stabile non la dev del ramo di sviluppo che potrebbe\n> non esserlo, se vuole partecipare allo sviluppo prende il ramo dev e\n> apporta su quello le sue modifiche (forkando o meno)\n>\n> —\n> Reply to this email directly or view it on GitHub\n> <https://github.com/FattiDiCookies/italianPrivacyPolicy/issues/73#issuecomment-117214148>\n> .\n>\n"}}