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 de documentação #286

Merged
merged 4 commits into from
Jun 24, 2023
Merged

Atualização de documentação #286

merged 4 commits into from
Jun 24, 2023

Conversation

HenriqueAJNB
Copy link
Contributor

@HenriqueAJNB HenriqueAJNB commented May 1, 2023

Este PR, quando aprovado, irá:

  • Acrescentar alterações para atualização da instalação do Python no Linux
  • Acrescentar alterações para atualização da instalação do Python no Windows

As documentações de instalação estavam bem defasadas, por isso a minha sugestão de atualização.

Eu sugiro também substituir alguns prints do terminal com o comando e sua respectiva saída em markdown, até para evitar publicar nomes e pastas pessoais.

adorilson
adorilson previously approved these changes Jun 24, 2023
Copy link
Collaborator

@adorilson adorilson left a comment

Choose a reason for hiding this comment

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

LGTM.

@adorilson adorilson requested a review from anadulce June 24, 2023 14:44
muriloviana
muriloviana previously approved these changes Jun 24, 2023
Copy link
Member

@muriloviana muriloviana left a comment

Choose a reason for hiding this comment

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

Deixei apenas uma leve sugestão no texto. No geral, ficou ótimo. Muito obrigado pela contribuição \o/

content/pages/instalacao-linux.md Outdated Show resolved Hide resolved
Co-authored-by: Murilo Viana <murilo.vianamo@gmail.com>
@adorilson adorilson dismissed stale reviews from muriloviana and themself via 14d10e3 June 24, 2023 23:16
@adorilson adorilson merged commit ffe1d5f into pythonbrasil:pelican Jun 24, 2023
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.

3 participants