search

php / doc-it

Italian translation of the PHP documentation
10 stars 10 forks source link

Prima pull request #2

Closed lucalzetta closed 3 years ago

lucalzetta commented 3 years ago

Prova di pull request, speriamo bene

lucalzetta commented 3 years ago

Non riesco a capire se i file che modifico vengono caricati o no sul documento master php/doc-it, se sono controllati e se hanno l'effetto che dovrebbero avere, modificare lo stato della traduzione.

DavidePastore commented 3 years ago

Ciao @lucalzetta . Una "pull request" è identificata come una "proposta di modifiche" che deve essere approvata ed accettata per poter essere inclusa nel branch principale (master). Quando avrai terminato le modifiche procederò a fare una review e eventualmente ad effettuare il merge.

DavidePastore commented 3 years ago

@lucalzetta Ovviamente fammi sapere se vuoi fermarti qui. In ogni caso in futuro potrai creare altre pull request per poter aggiornare o modificare altri file.

lucalzetta commented 3 years ago

Assolutamente vado avanti. Vedo che anche i file a cui ho semplicemente apposto un commit sono visibili in questa request. Dunque deve rimanere aperta fino alla fine del lavoro o, quantomeno, di un ciclo piuttosto importante. Io smanetto un po, se faccio confusione fammelo sapere. Ciao e grazie dell'assistenza

lucalzetta commented 3 years ago

Super, comincio a capire! Quanta forza, è come guidare il trattore :)

DavidePastore commented 3 years ago

Ciao @lucalzetta . C'è un motivo per cui hai chiuso la pull request?

lucalzetta commented 3 years ago

No volevo provare ad aprirne un'altra per testare la funzionalità. Se ne tengo due aperte genero qualche conflitto?

Il giorno mer 24 mar 2021 alle ore 09:42 Davide Pastore < @.***> ha scritto:

Ciao @lucalzetta https://github.com/lucalzetta . C'è un motivo per cui hai chiuso la pull request?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/php/doc-it/pull/2#issuecomment-805613584, or unsubscribe https://github.com/notifications/unsubscribe-auth/APE3ZWERA7ON24FXEFSZQIDTFGQYBANCNFSM4ZSXWENA .

DavidePastore commented 3 years ago

@lucalzetta puoi aprire quante pull request vuoi senza alcun problema. Tieni conto però che di solito una pull request dovrebbe essere quanto più atomica possibile, in modo da evitare grandi cambiamenti tutti insieme. Un esempio potrebbe essere quello di crearne una con la traduzione di un unico file o di un'intera sezione. In genere si tende ad evitare di avere pull request con tanti file modificati al suo interno.

lucalzetta commented 3 years ago

Buongiorno a tutti, ho modificato alcuni documenti, creati altri, ma non vedo gli effetti delle modifiche sul ramo principale https://github.com/php/doc-it.git. Dipende da me? Come posso fare a collegare le modifiche che ho fatto con quel ramo?

Ciao buona giornata

Il giorno mer 24 mar 2021 alle ore 11:43 Davide Pastore < @.***> ha scritto:

@lucalzetta https://github.com/lucalzetta puoi aprire quante pull request vuoi senza alcun problema. Tieni conto però che di solito una pull request dovrebbe essere quanto più atomica possibile, in modo da evitare grandi cambiamenti tutti insieme. Un esempio potrebbe essere quello di crearne una con la traduzione di un unico file o di un'intera sezione. In genere si tende ad evitare di avere pull request con tanti file modificati al suo interno.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/php/doc-it/pull/2#issuecomment-805707420, or unsubscribe https://github.com/notifications/unsubscribe-auth/APE3ZWGEMT33XYQEOZ67J6DTFG6VPANCNFSM4ZSXWENA .

DavidePastore commented 3 years ago

@lucalzetta , le pull request che crei devono essere accettate ed incluse sulla repository principale da un utente che ha permesso di scrittura (io ad esempio). Questi i passi che ti consiglio di fare:

lucalzetta commented 3 years ago

Ho tradotto alcuni file e passato 4 pull request, sto iniziando a imparare, piano piano. Vedo le mie pull su php/doc-it però non ho nessun riscontro che siano viste da altri. Voi le vedete? Vanno bene così? La prima è piuttosto ampia e comprende altre tre pull nidificate create per sbaglio ma contenenti modifiche. Le altre sono migliorato e sono più coerenti. Potreste dargli un'occhiata? Grazie, scusate la confusione. Ciao

Il giorno lun 29 mar 2021 alle ore 10:25 Davide Pastore < @.***> ha scritto:

@lucalzetta https://github.com/lucalzetta , le pull request che crei devono essere accettate ed incluse sulla repository principale da un utente che ha permesso di scrittura (io ad esempio). Questi i passi che ti consiglio di fare:

  • crea una nuova pull request quanto più atomica possibile;
  • congelala, evitando di aggiungere nuovi commit sulla stessa;
  • non chiuderla, altrimenti non verrà presa in considerazione.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/php/doc-it/pull/2#issuecomment-809179903, or unsubscribe https://github.com/notifications/unsubscribe-auth/APE3ZWH7N55MOWUPEYPELQTTGBBR3ANCNFSM4ZSXWENA .

DavidePastore commented 3 years ago

Ciao @lucalzetta. Ho visto le tue pull request e purtroppo non ho trovato il tempo per dedicarmi alle revisioni delle stesse. In ogni caso, dato che hai incluso decine di file da controllare e ho notato un impegno molto importante da parte tua, prevedo di fare un'analisi di uno o due file ed in caso darti dei suggerimenti per migliorarle. Dopo un paio di cicli di questo tipo, procederemo alla creazione di un account che ti permetterà di essere completamente autonomo.

lucalzetta commented 3 years ago

Grazie, però così non smetterò di tormentarvi per capire come proseguire 😀

Il giorno gio 8 apr 2021 alle ore 10:22 Davide Pastore < @.***> ha scritto:

Ciao @lucalzetta https://github.com/lucalzetta. Ho visto le tue pull request e purtroppo non ho trovato il tempo per dedicarmi alle revisioni delle stesse. In ogni caso, dato che hai incluso decine di file da controllare e ho notato un impegno molto importante da parte tua, prevedo di fare un'analisi di uno o due file ed in caso darti dei suggerimenti per migliorarle. Dopo un paio di cicli di questo tipo, procederemo alla creazione di un account che ti permetterà di essere completamente autonomo.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/php/doc-it/pull/2#issuecomment-815559820, or unsubscribe https://github.com/notifications/unsubscribe-auth/APE3ZWFHIY2JZDVDYIDKVKTTHVRSZANCNFSM4ZSXWENA .

DavidePastore commented 3 years ago

@lucalzetta, nessun problema. Continueremo a darti supporto in ogni caso. 😄