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 textos na documentação oficial do projeto #92

Merged
merged 13 commits into from
Feb 23, 2024
Merged

Conversation

trevineju
Copy link
Member

No description provided.

@ogecece ogecece force-pushed the reescrita-docs branch 2 times, most recently from 3e155a2 to 9913d3b Compare December 22, 2023 19:05
@trevineju trevineju force-pushed the reescrita-docs branch 2 times, most recently from ac74602 to d067d62 Compare January 10, 2024 23:17
@trevineju trevineju force-pushed the reescrita-docs branch 2 times, most recently from 2a9f058 to 4bbb9e8 Compare February 6, 2024 13:31
@trevineju
Copy link
Member Author

PR está pronto. A preview está em: https://querido-diario--92.org.readthedocs.build/pt-br/92/index.html

Só fiquei com a impressão que o texto de apresentação no index poderia ser mais impressionante, por ser a página inicial, e deixar pro conheca uma redação mais formal. Mas também não tive ideias boas de escrita.

Copy link
Member

@ogecece ogecece left a comment

Choose a reason for hiding this comment

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

Sugeri poucas mudanças estruturais. Pra mim o guia de contribuição tá coerente para contextualizar pessoas mais iniciantes e objetivo o suficiente pra não cansar demais quem já entende. Não acho que precisamos levar material pra parte do "entendendo o projeto".

Ficou excelente!

docs/pt_BR/source/entendendo/conheca.rst Outdated Show resolved Hide resolved
docs/pt_BR/source/entendendo/conheca.rst Outdated Show resolved Hide resolved
docs/pt_BR/source/entendendo/conheca.rst Outdated Show resolved Hide resolved
docs/pt_BR/source/entendendo/conheca.rst Outdated Show resolved Hide resolved
docs/pt_BR/source/entendendo/arquitetura.rst Outdated Show resolved Hide resolved
docs/pt_BR/source/contribuindo/raspadores.rst Outdated Show resolved Hide resolved
docs/pt_BR/source/contribuindo/raspadores.rst Outdated Show resolved Hide resolved
docs/pt_BR/source/contribuindo/raspadores.rst Outdated Show resolved Hide resolved
docs/pt_BR/source/contribuindo/raspadores.rst Outdated Show resolved Hide resolved
docs/pt_BR/source/contribuindo/raspadores.rst Outdated Show resolved Hide resolved
@jvanz jvanz self-requested a review February 10, 2024 14:36
Copy link
Member

@ogecece ogecece left a comment

Choose a reason for hiding this comment

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

Subi umas modificações na página de arquitetura (fico aguardando apenas a nova imagem do fluxo das etapas) e também no embed do vídeo no uso da API. Vê aí o que acha.

Fora isso, acho que só tenho um questão a mais pra pedir. Tá 100/100

docs/pt_BR/source/entendendo/projetos-relacionados.rst Outdated Show resolved Hide resolved
docs/pt_BR/source/contribuindo/raspadores.rst Outdated Show resolved Hide resolved
docs/pt_BR/source/contribuindo/raspadores.rst Outdated Show resolved Hide resolved
@trevineju
Copy link
Member Author

@jvanz você pretende dar alguma review por aqui?

Copy link
Member

@ogecece ogecece left a comment

Choose a reason for hiding this comment

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

Tá shou! Só dá uma organizada nos commits e podemos mesclar.

_

@trevineju trevineju merged commit d65f1cc into main Feb 23, 2024
1 check passed
@trevineju trevineju deleted the reescrita-docs branch February 23, 2024 22:45
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.

Adicionar referências de estudo sobre o QD e raspagem | Add study references about QD and scraping
2 participants