-
Notifications
You must be signed in to change notification settings - Fork 100
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
Translation of adding-search.md #149
Translation of adding-search.md #149
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Excelente tradução, @sosolidkk! Muito boa mesmo.
Elenquei apenas alguns typos que você cometeu, e uma pequena sugestão de uma frase que eu achei confusa, e adoraria que você desse tua opinião.
Todas as alterações realizadas já! Obrigado pelo review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ótima tradução, obrigado por enviar
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ótimo! Muito obrigado @sosolidkk! :sparkes:
Qual o objetivo dessa pull request?
Qual arquivo foi traduzido/corrigido?
adding-search.md
Algum comentário em relação a tradução?
Em alguns momentos fiquei em dúvida entre utilizar pesquisa ou busca na hora de traduzir search. Tentei alternar para evitar repetição.