Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Data types#243

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged

Conversation

jonnathan-ls
Copy link
Contributor

Data types

📝Goal

Article update

☑ What was done?

  • Pending text translation

renatocfrancisco reacted with laugh emoji
@javascript-translate-bot

Error: the article already translated in the Progress Issue#1⁉️

@jonnathan-ls
Copy link
ContributorAuthor

Error: the article already translated in the Progress Issue#1⁉️
#243 (comment)javascript-translate-bot

There is still text that has not been translated.

@odsantos
Copy link
Contributor

Error: the article already translated in the Progress Issue#1 interrobang
#243 (comment)javascript-translate-bot

There is still text that has not been translated.

Hi,

Will you translate the missing parts, including solution and task?
It would be helpful to review that as lesson, the whole translated article as well as the rest.

Trying to know your plan regarding to this lesson.

Thanks, Osvaldo.

jonnathan-ls reacted with thumbs up emoji

Copy link

@renatocfranciscorenatocfrancisco left a comment

Choose a reason for hiding this comment

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

In line 73, the english sentence still in the file.
Moreover, good job!

jonnathan-ls reacted with heart emoji
@jonnathan-ls
Copy link
ContributorAuthor

jonnathan-ls commentedAug 6, 2023•
edited
Loading

Olá@renatocfrancisco,

Obrigado pela observação, não tinha percebido ❤️

Olá@odsantos,

Will you translate the missing parts, including solution and task?
It would be helpful to review that as lesson, the whole translated article as well as the rest.

O PR contempla toda tradução do artigo, com exceção da linha 73 como citado pelo Renato
Adicionei um novo commit com o ajuste.

Por desencargo, também atualizei com a master do repositório original.

Poderia adicionar sua revisão a este PR?

renatocfrancisco reacted with thumbs up emoji

@odsantos
Copy link
Contributor

Hi@jonnathan-ls , sim. Logo que possivel. Obrigado.

jonnathan-ls reacted with heart emoji

@nazarepiedady
Copy link
Member

@peruibeloko,@jonnathan-ls, what is missing in this pull request?

@jonnathan-ls
Copy link
ContributorAuthor

... what is missing in this pull request?

The review by two maintainers is missing, I will not contribute with the review, as the PR is precisely my own, there is no point in reviewing it myself

@nazarepiedady Could you add your review?

nazarepiedady reacted with thumbs up emoji

Copy link
Member

@nazarepiedadynazarepiedady left a comment

Choose a reason for hiding this comment

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

We could improve it more ahead, but for now, it is good enough to be approved.

@nazarepiedady
Copy link
Member

@jonnathan-ls, now we need one more review. It should take some time because we are running under forces.

jonnathan-ls reacted with heart emoji

@nazarepiedady
Copy link
Member

@peruibeloko, are you available to review this pull request?

Copy link
Contributor

@peruibelokoperuibeloko left a comment

Choose a reason for hiding this comment

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

Duas mudanças rápidas e está aprovado

Em JavaScript,o tipo"number"não pode representar valores inteiros maiores que<code>(2<sup>53</sup>-1)</code> (que é`9007199254740991`) ou menores que<code>-(2<sup>53</sup>-1)</code>para negativos. É uma limitação técnica causada por sua representação interna.

For most purposes that's quite enough, but sometimes we need really big numbers, e.g. for cryptography or microsecond-precision timestamps.
Para a maioria dos propósitos, isso é o suficiente, mas às vezes precisamos de números realmente grandes, por exemplo, para criptografia ou carimbos de data/hora com precisão de microssegundos.
Copy link
Contributor

Choose a reason for hiding this comment

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

This section is duplicated at line 75

Copy link
ContributorAuthor

Choose a reason for hiding this comment

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

Removed

```

You can check [*MDN* BigInt compatibility table](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/BigInt#Browser_compatibility) to know which versions of a browser are supported.
Você pode verificar a [tabela de compatibilidadedo*MDN* BigInt](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/BigInt#browser_compatibility) para saber quais versões de um navegador são suportadas.
Copy link
Contributor

Choose a reason for hiding this comment

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

Você pode verificar a [tabela de compatibilidade do *MDN* sobre BigInt]

ou

Você pode verificar a [tabela de compatibilidade do BigInt na *MDN*]

Copy link
ContributorAuthor

Choose a reason for hiding this comment

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

Done

@javascript-translate-bot

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

@nazarepiedady
Copy link
Member

nazarepiedady commentedApr 9, 2024•
edited
Loading

@jonnathan-ls, could you consider the reviews did?

@jonnathan-ls
Copy link
ContributorAuthor

jonnathan-ls commentedApr 14, 2024•
edited
Loading

... could you consider the reviews did?

/done@nazarepiedady

I applied the requests noted by@peruibeloko, see commit4a7029e. And I also added pending content that I noticed when accessing the article from the official English repositorybigint-type session, see commit7cafcbb

@javascript-translate-botjavascript-translate-bot requested a review froma teamApril 14, 2024 17:39
Copy link
Member

@nazarepiedadynazarepiedady left a comment

Choose a reason for hiding this comment

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

It seems good enough to be approved.

Copy link
Contributor

@peruibelokoperuibeloko left a comment

Choose a reason for hiding this comment

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

LGTM

nazarepiedady and renatocfrancisco reacted with thumbs up emoji
@nazarepiedadynazarepiedady merged commite9d2d8c intojavascript-tutorial:masterApr 16, 2024
@javascript-translate-bot

Error: the article already translated in the Progress Issue#1⁉️

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@nazarepiedadynazarepiedadynazarepiedady approved these changes

+2 more reviewers

@renatocfranciscorenatocfranciscorenatocfrancisco left review comments

@peruibelokoperuibelokoperuibeloko approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@jonnathan-ls@javascript-translate-bot@odsantos@nazarepiedady@peruibeloko@renatocfrancisco
[8]ページ先頭
©2009-2025 Movatter.jp