-
Notifications
You must be signed in to change notification settings - Fork 70
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
Finished translation of Introduction to cypress - Chains of Commands #47
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.
@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 --> |
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.
Porque você adicionou essa linha?
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.
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?
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.
Você não precisa ignorar o erro, você traduz e tenta seguir o padrão das regras. Entende?
@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 |
|
@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 😄 |
@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! |
@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? Outra coisa, me procura no Telegram caso você queira tirar dúvidas pontuais sobre o projeto,etc. meu username é @pedrohyvo |
Closes #13