Prima pull request #2
Conversation
le righe da 306 a 311 sono confuse, indicano un rapporto di compressione ma sono scritte male, andrebbero rifatte anche in inglese, secondo me
|
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. |
|
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. |
|
@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. |
Traduzione di migration56.xml e creazione di una pull request di prova delle funzionalità di git hub
…uzione Update migration56.xml
Traduzione di migration70.xml header da modificare
Aggiunto riga all'header con nome del traduttore.
|
Assolutamente vado avanti. |
| livelli speciali di compressione: 0 (per non eseguire nessuna compressione), | ||
| e -1 (valore interno di default per zlib), 6 � un valore predefinito. | ||
|
|
||
| <parameter>window</parameter> � il logaritmo in base -2 del ciclo di compressione della finestra?. |
There was a problem hiding this comment.
Da qui alla riga 311 il discorso è poco chiaro, secondo me andrebbe rivisto anche in inglese
|
Super, comincio a capire! |
|
Ciao @lucalzetta . C'è un motivo per cui hai chiuso la pull request? |
|
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
<#2 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/APE3ZWERA7ON24FXEFSZQIDTFGQYBANCNFSM4ZSXWENA>
.
|
|
@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. |
|
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
<#2 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/APE3ZWGEMT33XYQEOZ67J6DTFG6VPANCNFSM4ZSXWENA>
.
|
|
@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:
|
|
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
<#2 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/APE3ZWH7N55MOWUPEYPELQTTGBBR3ANCNFSM4ZSXWENA>
.
|
|
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. |
|
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
<#2 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/APE3ZWFHIY2JZDVDYIDKVKTTHVRSZANCNFSM4ZSXWENA>
.
|
|
@lucalzetta, nessun problema. Continueremo a darti supporto in ogni caso. 😄 |
Prova di pull request, speriamo bene