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

Atualização do capítulo 2 com base na versão oficial #188

Merged
merged 2 commits into from
May 18, 2023

Conversation

mcquintao
Copy link

Atualização do conteúdo do capítulo 2

  • Atualização do conteúdo, seguindo o livro oficial.
  • Adição de snips nos blocos de código e do ferris (quando o código não compila)
  • Atualização da biblioteca rand de 0.4.15 para 0.8.5 (modificado .gen_range(1, 101) para .gen_range(1..=100))
  • Remoção do --bin para gerar o projeto binário com Cargo (padrão)
  • Correção dos links quebrados para sessões do próprio livro ou para Crates e Docs externas.

e outras coisas menores...

Chequei o conteúdo e ortografia algumas vezes, caso tenha deixado passar algo, só comentar!
Tive que me utilizar de recursos adaptativos (aka gambiarra) para adicionar a imagem do ferris próximo ao bloco de código. Não ficou exatamente dentro do bloco, igual na versão em ingles, mas se algúem souber como corrigir, só me falar!

PS: É meu primeiro PR da vida então qualquer dica/modificações/observações agradeço d+.

Revisem com carinho e desculpem-me por qalquer problema!
Abraços!

Copy link
Member

@marioidival marioidival left a comment

Choose a reason for hiding this comment

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

Obrigado pela sua ajuda na tradução do cap!

Tenho 2 pontos de melhoria, caso ache necessario.

Outra coisa é que parece que tem o cap escrito duas vezes :D

@mcquintao
Copy link
Author

@marioidival obrigado pela revisão.
Onde tem o cap escrito duas vezes? Não vi rsrs

@mcquintao mcquintao requested a review from marioidival May 18, 2023 00:54
@marioidival marioidival merged commit 43b2c1e into rust-br:master May 18, 2023
@marioidival
Copy link
Member

Muito obrigado @mcquintao

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