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 as instruções sobre como criar spiders (#149) #156

Closed
wants to merge 5 commits into from

Conversation

augusto-herrmann
Copy link
Contributor

atualiza o README em português e inglês, explicando como criar spiders com a nova interface de programação, no diretório web, a partir da classe BaseCovid19Spider.

@endersonmaia endersonmaia requested a review from turicas June 4, 2020 18:22
@endersonmaia
Copy link
Collaborator

acho uma boa oportunidade para já remover os scripts antigos e referências a eles :

  • corona_??_spider.py
  • collect.sh (existe referência no README.md)
  • Makefile target docker-collect

eu não sei avaliar qual impacto disto, se estão sendo usados, e nem se a estrutura que está em ./web já está usável de forma automatizada, parece-me que ainda depende de executar o script ./web.sh que também não tem instruções neste PR

@endersonmaia endersonmaia added the documentation Improvements or additions to documentation label Jun 4, 2020
@augusto-herrmann
Copy link
Contributor Author

Sim, depois que mandei o PR pensei em incluir mesmo as instruções para rodar o web.sh. Por outro lado, já começa a ficar muito texto para apenas uma seção do README. O que acha de separar as instruções para criar scrapers em um documento específico?

acho uma boa oportunidade para já remover os scripts antigos e referências a eles :

* `corona_??_spider.py`

* `collect.sh` (existe referência no `README.md`)

* `Makefile` target `docker-collect`

Sim. Essas coisas que não são documentação podem ser feitas em outro PR. A parte do collect.sh poderia já mudar, mas não sei o que escrever, pois fiquei sabendo do modo de funcionamento dos novos scraper de uma forma quase que uma engenharia reversa. Vou deixar isso para outra pessoa fazer, alguém que conheça melhor o seu funcionamento que eu. 🙂

@endersonmaia
Copy link
Collaborator

Sim, depois que mandei o PR pensei em incluir mesmo as instruções para rodar o web.sh. Por outro lado, já começa a ficar muito texto para apenas uma seção do README. O que acha de separar as instruções para criar scrapers em um documento específico?

acho válido sim, o README.md tá ficando poluído mesmo

eu tinha feito um PR que foi rejeitado, colocando tudo que é *.md numa pasta ./docs

o motivo da rejeição foi que alguns documentos já eram linkados diretamente

talvez os novos documentos já pudessem ser criados dentro de ./docs

@endersonmaia endersonmaia linked an issue Jun 7, 2020 that may be closed by this pull request
@Bonifacio2
Copy link
Collaborator

Massa esse PR! Eu acabei escrevendo um spider no modelo antigo porque não sabia que existia um novo.

@augusto-herrmann
Copy link
Contributor Author

Acrescentei as instruções para subir o serviço de scaping com o ./web.sh.

A parte do collect.sh eu resolvi não mexer. Talvez o pessoal ainda queira automatizar e mudar esse arquivo para coletar pelos scrapers novos.

@augusto-herrmann
Copy link
Contributor Author

@turicas, foi feito o rebase para tornar novamente possível o merge.

@augusto-herrmann
Copy link
Contributor Author

Lembrando o PR, @turicas.

@augusto-herrmann
Copy link
Contributor Author

@turicas, podemos fazer rebase/merge?

@augusto-herrmann
Copy link
Contributor Author

ping @turicas

@augusto-herrmann
Copy link
Contributor Author

Estou desistindo deste PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Atualizar README.md com instruções para criar scrapers
3 participants