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 Capítulo 9 #468

Open
wants to merge 3 commits into
base: 3rd
Choose a base branch
from
Open

Conversation

biliesilva
Copy link

@biliesilva biliesilva commented Jul 14, 2021

Finalizada a alteracao para o branch correto da 3rd.

#408

Copy link
Contributor

@joaostein joaostein left a comment

Choose a reason for hiding this comment

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

@bilianesilva -- primeira revisão iniciada com alguns itens.


{{quote {author: "Jamie Zawinski", chapter: true}
Copy link
Contributor

Choose a reason for hiding this comment

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

Reparei que você removeu do arquivo todos os metadados (tudo que está entre {{}}). Você poderia, por gentileza, manter a estrutura do arquivo original, sem remover essas informações?

Copy link
Author

Choose a reason for hiding this comment

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

Eu acredito que tenha sido a minha extensão que uso no VScode. Vou dar uma olhada e consertar.

09_regexp.md Show resolved Hide resolved
Copy link
Contributor

@joaostein joaostein left a comment

Choose a reason for hiding this comment

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

@bilianesilva deixei alguns comentários para ajuste. Lembre-se de que estamos apenas traduzindo o conteúdo e queremos manter a estrutura o máximo possível igual ao arquivo original. Isso inclui os blocos de códigos também. Qualquer coisa só mandar mensagem que vamos conversando.


{{quote {author: "Jamie Zawinski", chapter: true}

Some people, when confronted with a problem, think 'I know, I'll use
regular expressions.' Now they have two problems.
> "Algumas pessoas, quando confrontadas com um problema, pensam "Eu sei, terei que usar expressões regulares." Agora elas têm dois problemas.
Copy link
Contributor

Choose a reason for hiding this comment

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

Remover >

Yuan-Ma said, 'When you cut against the grain of the wood, much
strength is needed. When you program against the grain of the problem,
much code is needed.'
> "Yuan-Ma disse, 'Quando você serra contra o sentido da madeira, muita força será necessária. Quando você programa contra o sentido do problema, muito código será necessário'
Copy link
Contributor

Choose a reason for hiding this comment

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

Remover >


```
```js
Copy link
Contributor

Choose a reason for hiding this comment

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

remover js

Isso acontece em todos os blocos de código. Vamos manter o máximo possível igual o original.

| `\W` | A nonalphanumeric character
| `\S` | A nonwhitespace character
| `.` | Any character except for newline
- \d caracteres numéricos
Copy link
Contributor

Choose a reason for hiding this comment

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

você removeu as |, os `` e alterou o espaçamento. Vamos manter a estrutura igual o original, apenas traduza os textos.


{{index "plus character", repetition, "+ operator"}}
{{index "plus character", repetition, `+ operator`}}
Copy link
Contributor

Choose a reason for hiding this comment

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

Manter como o original, você trocou aspas pelo `

["earth bed", "learning ape", "BEET"]);
verify(
/.../,
["Siebentausenddreihundertzweiundzwanzig"],
Copy link
Contributor

Choose a reason for hiding this comment

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

mantenha como o original


verify(
/.../,
["red platypus", "wobbling nest"],
Copy link
Contributor

Choose a reason for hiding this comment

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

mantenha como o original

}
// Ignore unfinished exercises
if (regexp.source == "...") return;
for (let str of yes)
Copy link
Contributor

Choose a reason for hiding this comment

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

mantenha como o original

if (!number.test(str)) {
console.log(`Failed to match '${str}'`);
}
for (let str of [
Copy link
Contributor

Choose a reason for hiding this comment

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

mantenha como o original

if (number.test(str)) {
console.log(`Incorrectly accepted '${str}'`);
}
for (let str of ["1a", "+-1", "1.2.3", "1+1", "1e4.5", ".5.", "1f5", "."]) {
Copy link
Contributor

Choose a reason for hiding this comment

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

mantenha como o original

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