Skip to content

Comments

install.unix page#49

Merged
DavidePastore merged 5 commits intophp:masterfrom
fabiosal:install_unix
Feb 24, 2026
Merged

install.unix page#49
DavidePastore merged 5 commits intophp:masterfrom
fabiosal:install_unix

Conversation

@fabiosal
Copy link
Contributor

install.unix page translation

@DavidePastore DavidePastore added the enhancement New feature or request label Feb 10, 2025
@DavidePastore
Copy link
Member

Ciao @fabiosal. Thanks for your PR and the interest you show in translating the PHP documentation. The most important note is that you need to use the same number of lines of the original file so it will be easier to maintain it in the future. Mind that you can still compare this version to the corresponding English version by following this link.

Could you please ensure that all the sections have the same number of lines in your proposal? The English one has 199 lines, we should have 200 because we have two comments on top of the file. Thanks in advance!

Copy link
Member

@DavidePastore DavidePastore left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Check this comment.

@fabiosal
Copy link
Contributor Author

@DavidePastore, I modified the text to remove some lines in order to fit on 200 lines as the english revision.

Copy link
Member

@DavidePastore DavidePastore left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Come specificato qui, le righe devono avere una corrispondenza nella versione inglese e italiana per permettere una più facile manutenzione ed un aggiornamento più veloce.

@DavidePastore
Copy link
Member

Ciao, @fabiosal. Hai bisogno di una mano per ultimare il tuo lavoro su questo file?

@DavidePastore
Copy link
Member

Ciao @fabiosal. Ho apportato le modifiche io stesso e ho effettuato il merge della PR. Grazie per il tuo contributo.

@DavidePastore DavidePastore merged commit b7e675b into php:master Feb 24, 2026
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants