Skip to content

Conversation

@peruibeloko
Copy link
Contributor

No description provided.

Copy link
Member

@nazarepiedady nazarepiedady left a comment

Choose a reason for hiding this comment

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

It is good enough to be considered for applying.

@nazarepiedady
Copy link
Member

@jonnathan-ls, we need your help here, do you mind reviewing this pull request?

@nazarepiedady
Copy link
Member

@danilolmc, do you have some time to review this pull request and approve it if it is good?

Copy link
Contributor

@gabifs gabifs left a comment

Choose a reason for hiding this comment

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

@nazarepiedady @peruibeloko, encontrei essas alterações nos exemplos, que não me parecem corretas. Se puderem revisar novamente


```js run
let regexp = new RegExp("\d\.\d");
let regexp = new RegExp("d.d");
Copy link
Contributor

Choose a reason for hiding this comment

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

Me parece que aqui houve uma alteração no exemplo

Suggested change
let regexp = new RegExp("d.d");
let regexp = new RegExp("\d\.\d");

Copy link
Member

Choose a reason for hiding this comment

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

@gabifs, the priority is to conclude the translation first, and then update the content to be current as the English original content.


```js run
alert("\d\.\d"); // d.d
alert("d.d"); // d.d
Copy link
Contributor

Choose a reason for hiding this comment

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

Aqui aconteceu de novo:

Suggested change
alert("d.d"); // d.d
alert("\d\.\d"); // d.d

@gabifs gabifs merged commit c3e18a3 into javascript-tutorial:master Nov 24, 2025
1 check passed
@peruibeloko peruibeloko deleted the 09/07 branch December 7, 2025 18:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants