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

Object references and copying #408

Merged

Conversation

danilolmc
Copy link
Contributor

No description provided.

Copy link
Contributor

@jonnathan-ls jonnathan-ls left a comment

Choose a reason for hiding this comment

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

Olá @danilolmc,

Eu tentei empurrar para sua branch algumas alterações simples de gramática que acabei fazendo enquanto realizava a revisão deste PR.

Porém, seu fork/user deve estar protegido para receber commits de outros usuários, acabei recebendo error de permission denied:

! [remote rejected] pr/408 -> update-pt-object-copy (permission denied)
error: failed to push some refs to 'https://github.com/danilolmc/pt.javascript.info.git'

Nesse sentido, registro abaixo as alterações que eu ia adicionar ao seu PR, poderia verificar as solicitações a seguir?

image

image

1-js/04-object-basics/02-object-copy/article.md Outdated Show resolved Hide resolved
1-js/04-object-basics/02-object-copy/article.md Outdated Show resolved Hide resolved
1-js/04-object-basics/02-object-copy/article.md Outdated Show resolved Hide resolved
1-js/04-object-basics/02-object-copy/article.md Outdated Show resolved Hide resolved
1-js/04-object-basics/02-object-copy/article.md Outdated Show resolved Hide resolved
1-js/04-object-basics/02-object-copy/article.md Outdated Show resolved Hide resolved
1-js/04-object-basics/02-object-copy/article.md Outdated Show resolved Hide resolved
1-js/04-object-basics/02-object-copy/article.md Outdated Show resolved Hide resolved
@javascript-translate-bot

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

@danilolmc
Copy link
Contributor Author

/done

@javascript-translate-bot javascript-translate-bot requested a review from a team December 20, 2023 02:40
@nazarepiedady nazarepiedady merged commit 769acd2 into javascript-tutorial:master Dec 20, 2023
1 check passed
@danilolmc danilolmc deleted the update-pt-object-copy branch December 21, 2023 16:50
@danilolmc danilolmc changed the title Updates Object references and copying article Object references and copying Dec 22, 2023
@jonnathan-ls
Copy link
Contributor

jonnathan-ls commented Dec 23, 2023

@nazarepiedady como fez o merge, entendo que adicionou sua revisão como aprovada como a segunda revisora, creio ser válido deixar registrado que o PR esta aprovado, conforme seção abaixo

image

Após o merge não consigo realizar a atualização, mas deixo a observação para próximos PRs

A aprovação de dois revisores corrobora com a orientação deixa pelo Ilkan #1

image

@nazarepiedady
Copy link
Member

@jonnathan-ls, entendo, considerarei este ponto da próxima vez que precisar combinar as modificações enviadas ao repositório principal. Na primeira leitura que tinha feito pareceu-me bem o trabalho feito, por esta razão fiz a combinação, não volta acontecer.

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.

4 participants