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

Finished translation of Introduction to cypress - Chains of Commands #47

Merged

Conversation

edmundobiglia
Copy link
Contributor

@edmundobiglia edmundobiglia commented Feb 28, 2021

Closes #13

Copy link
Owner

@pedrohyvo pedrohyvo left a comment

Choose a reason for hiding this comment

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

@edmundobiglia excelente trabalho Edmundo, deixei apenas alguns comentários antes de fazer o merge

comandos sejam encadeados neles (pressupondo que eles esperam um sujeito do DOM), tais como [`.click()`](https://docs.cypress.io/api/commands/click.html)
ou até mesmo [`cy.contains()`](https://docs.cypress.io/api/commands/contains.html) novamente.

<!-- markdownlint-disable MD026 -->
Copy link
Owner

Choose a reason for hiding this comment

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

Porque você adicionou essa linha?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Adicionei essa linha porque estava sinalizando como erro o fato de ter pontuação no título, porém a pontuação está no título em inglês. Nesse caso posso tirar a linha e ignorar o erro?

Copy link
Owner

Choose a reason for hiding this comment

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

Você não precisa ignorar o erro, você traduz e tenta seguir o padrão das regras. Entende?

@pedrohyvo pedrohyvo added the documentation Improvements or additions to documentation label Mar 1, 2021
@edmundobiglia
Copy link
Contributor Author

@pedrohyvo obrigado, Pedro! Como faço agora? Faço as correções e depois faço outro PR? (Desculpe, esse é meu primeiro PR, então não estou totalmente familiarizado com o processo.)

@pedrohyvo
Copy link
Owner

@pedrohyvo obrigado, Pedro! Como faço agora? Faço as correções e depois faço outro PR? (Desculpe, esse é meu primeiro PR, então não estou totalmente familiarizado com o processo.)

@edmundobiglia você faz as suas correções na mesma branch do PR, atualiza com a master e depois sobe suas alterações aqui. Quando você fizer o commit ele já atualiza automaticamente aqui, só você me mencionar que reviso novamente :D Qualquer dúvida, só perguntar

@edmundobiglia
Copy link
Contributor Author

@pedrohyvo obrigado, Pedro! Como faço agora? Faço as correções e depois faço outro PR? (Desculpe, esse é meu primeiro PR, então não estou totalmente familiarizado com o processo.)

@edmundobiglia você faz as suas correções na mesma branch do PR, atualiza com a master e depois sobe suas alterações aqui. Quando você fizer o commit ele já atualiza automaticamente aqui, só você me mencionar que reviso novamente :D Qualquer dúvida, só perguntar

@edmundobiglia
Copy link
Contributor Author

@pedrohyvo Enviei as atualizações. Agora não passou na verificação porque retirei a linha que sinaliza títulos/subtítulos com pontuação no final. Tudo bem?

@pedrohyvo pedrohyvo reopened this Mar 2, 2021
@pedrohyvo
Copy link
Owner

pedrohyvo commented Mar 2, 2021

@pedrohyvo Enviei as atualizações. Agora não passou na verificação porque retirei a linha que sinaliza títulos/subtítulos com pontuação no final. Tudo bem?

@edmundobiglia nesse caso, eu acredito que você possa seguir as regras do markdown lint, você pode remover a pontuação ao final da linha, por exemplo ":" ou ".", tudo bem?

É normal você ter esse tipo de dúvida, já que é seu primeiro PR. Nos próximos vai fluir bem melhor 😄

@edmundobiglia
Copy link
Contributor Author

@pedrohyvo já fiz as correções e enviei de novo as alterações. Qualquer coisa, se for necessária alguma nova correção, me avisa!

@pedrohyvo
Copy link
Owner

@edmundobiglia parabéns pela sua primeira contribuição no projeto cara!!!! 🍰 Quero te pedir uma coisa, você pode comentar essa issue aqui falando que vai ficar com toda a página da Introdução ao Cypress, pode ser?

#22

Outra coisa, me procura no Telegram caso você queira tirar dúvidas pontuais sobre o projeto,etc. meu username é @pedrohyvo

@pedrohyvo pedrohyvo merged commit 30b8a85 into pedrohyvo:master Mar 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Core Concepts] Add Introduction to Cypress page - Chains of Commands
2 participants