Skip to content

Revisao cap4 #299

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.

Sign up for GitHub

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

Jump to bottom
Closed
yuriscosta-zz wants to merge 3 commits into from
Closed

Revisao cap4 #299

yuriscosta-zz wants to merge 3 commits into from

Conversation

@yuriscosta-zz
Copy link
Contributor

@yuriscosta-zz yuriscosta-zz commented Oct 13, 2015

No description provided.

SauloSilva
SauloSilva reviewed Oct 13, 2015
View reviewed changes
chapters/04-estruturas-de-dados.md
Copy link
Member

@SauloSilva SauloSilva Oct 13, 2015

Choose a reason for hiding this comment

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

Aqui vai algumas considerações:

  • Arrays ou array em itálico
  • Mudar "value.index e value[index]" para "value.index e value[index]"
  • index em itálico

@SauloSilva
Copy link
Member

SauloSilva commented Oct 13, 2015

@YuriStyl3 fico feliz e grato pela sua contribuição (=
Posso mergear seu PR (pull request) agora e depois se quiser fazer as alterações que sugeri, sinta se a vontade para abrir um outro PR com as devidas melhorias que comentei.

Por hora necessito que você atualize teu branch revisaoCap4 com a master e faça um push para que eu consiga fazer o merge do teu PR.

Muito obrigado

@yuriscosta-zz
Copy link
Contributor Author

yuriscosta-zz commented Oct 13, 2015

Tá, vou tentar, sou meio noob com github ainda... :/
mais tarde eu faço as alterações

@lucasfcosta
Copy link
Contributor

lucasfcosta commented Oct 13, 2015

@YuriStyl3 , se precisar de alguma ajuda aqui tem um videozinho bem breve sobre rebase:
https://www.youtube.com/watch?v=SxzjZtJwOgo

Não vi o histórico todo, mas pelo que o Saulo falou, se você fizer um rebase jogando seus commits depois da ponta do master, resolver o conflito e dar um push -f vai dar pra mergear.

Se precisar de ajuda avise aí seu problema, abraço!

@SauloSilva
Copy link
Member

SauloSilva commented Oct 16, 2015

@YuriStyl3 se precisar de ajuda é só falar.

joaostein added a commit to joaostein/eloquente-javascript that referenced this pull request Jan 5, 2016
@joaostein
Adiciona melhorias capítulo 04, até linha 205. Closes braziljs#299 and …
6f438bc 
…closes braziljs#309.
@joaostein joaostein closed this in 1f3ef04 Jan 17, 2016
joaostein added a commit that referenced this pull request Jan 17, 2016
@joaostein
Merge pull request #334 from joaostein/cap4
117ddf5 
Adiciona melhorias capítulo 04, até linha 205. Closes #299, closes #309.
joaostein added a commit to joaostein/eloquente-javascript that referenced this pull request Jan 17, 2016
@joaostein
Adiciona melhorias capítulo 04, até linha 205. Closes braziljs#299, c…
dc2716e 
…loses braziljs#309.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@yuriscosta-zz @SauloSilva @lucasfcosta @joaostein