Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tradução cURL. #257

Merged
merged 15 commits into from
Aug 14, 2023
Merged

Tradução cURL. #257

merged 15 commits into from
Aug 14, 2023

Conversation

EnzoGamaDS
Copy link
Contributor

No description provided.

@alfsb
Copy link
Member

alfsb commented Mar 13, 2023

Ola @EnzoGamaDS , fui fazer a revisão da sua alteração. As partes traduzidas parecem boas, mas tem blocos inteiros de texto a mais entre os arquivos originais e traduzidos.

Por exemplo, no arquivo curl-exec.xml está faltando um <note> e o <changelog>. O hash colocado no arquivo eu também não localizei.

(percebi, aqui, que a listagem de arquivos sem tag (http://doc.php.net/revcheck.php?p=misstags&lang=pt_br) não gera os hash, e infelizmente não tenho como consertar ou encaminhar isso tão cedo. Para obter os hashs corretos para esses arquivos sem revtag é preciso rodar um revcheck.php localmente, como descrito no guia de edição)

Peço que veja o vídeo que tem na home do wiki, onde mostro a comparação de arquivo original e traduzido.

Abaixo a lista de hashs esperados para esses arquivos co cURL, conforme o revcheck.php:

reference/curl/functions
curl-exec.xml | b7f8c11e56ff1c57a2993e2ed7e5c5ace18637fd | 2932 | 2303 | 629
curl-getinfo.xml | f9c4a68ef4f89e51e6d9b905ad3ddb6492386dd3 | 20726 | 9261 | 11465
curl-init.xml | b7f8c11e56ff1c57a2993e2ed7e5c5ace18637fd | 3544 | 2821 | 723
curl-multi-close.xml | 52dc204a77076e1404257cf39f179882b90b5780 | 3067 | 2575 | 492
curl-multi-getcontent.xml | 427ab27e525c67f731ea0ef15d3d89e5f87fe191 | 2161 | 1780 | 381
curl-multi-info-read.xml | b7f8c11e56ff1c57a2993e2ed7e5c5ace18637fd | 5161 | 5191 | -30
curl-multi-init.xml | e41806c30bf6975e452c0d4ce35ab0984c2fa68c | 3125 | 2771 | 354
curl-multi-remove-handle.xml | b7f8c11e56ff1c57a2993e2ed7e5c5ace18637fd | 2547 | 2151 | 396
curl-multi-select.xml | b7f8c11e56ff1c57a2993e2ed7e5c5ace18637fd | 2423 | 2029 | 394
curl-setopt-array.xml | 184f3f7bd45643cb80f828d0bb001991ec3a5fad | 3592 | 3848 | -256
curl-version.xml | 08a941584f00b659f595dfc6d108a73b2e6849a6 | 4064 | 3689 | 375

EN-Revision code
EN-Revision code
EN-Revision code
@EnzoGamaDS
Copy link
Contributor Author

Alterações realizadas no EN-Revision.

@alfsb
Copy link
Member

alfsb commented Mar 29, 2023

Oh legal, progressos. Ainda falta algumas coisas, e é o seguinte. Compare os arquivos:

Esses são os arquivos no original e a tradução no fork. Os arquivos tem número de linhas diferentes, e quantidade de tags diferentes. Para poder subir o arquivo como atualizado, teria de estar com as mesmas tags, na mesma ordem.

Não sei teve a chance de ver em vídeo essa explicação, onde mostra a sincronização do texto, mantendo a mesma indentação e quantidade de linhas. No vídeo https://user-images.githubusercontent.com/10078152/219964798-96dcae76-70e1-44d7-95ca-38d340b181c2.webm aparece isso por volta do minuto 5:00, onde apesar de usar o diff como guia para saber onde fazer as alterações, ainda assim é feita a rápida alternância entre arquivo original e traduzido, para detectar diferenças de conteúdo.

Veja e depois comente. Em última instância, talvez seja necessário retraduzir o arquivo inteiro, linha a linha, para daí ficar sincronizado com o original.

@alfsb
Copy link
Member

alfsb commented Aug 14, 2023

Mais de 3 meses em inatividade, pegando esse issue para eu continuar.

@alfsb alfsb merged commit ffa47e5 into php:master Aug 14, 2023
1 check passed
@alfsb
Copy link
Member

alfsb commented Aug 14, 2023

Acabei removendo um dos arquivos, que tinha alterações muito grandes para conseguir fazer chegar igual ao texto em inglês. Os demais eu fiz a revisão e acrescentei as partes faltantes e/ou diferentes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants