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

Add crawler for Canoas/RS #91

Closed
wants to merge 2 commits into from
Closed

Add crawler for Canoas/RS #91

wants to merge 2 commits into from

Conversation

lucasrcezimbra
Copy link
Contributor

Segue o crawler para o novo site de Canoas/RS com diários a partir de 29/05/2018, como discutido aqui #10.
Me atrapalhei com o Docker e não consegui testar muito bem, se alguém puder ajudar nisso. Sei q os arquivos foram baixados, mas não consegui acessar.

@jvanz jvanz added the spider Adiciona robô raspador para município(s) label Jul 10, 2020
@jvanz jvanz changed the base branch from master to main August 12, 2020 01:39
@vitorbaptista vitorbaptista self-requested a review October 10, 2020 16:49
Copy link
Contributor

@vitorbaptista vitorbaptista left a comment

Choose a reason for hiding this comment

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

Eu atualizei o código com o que está no master e corrigi o caso de dias sem diário oficial. O código está ótimo, pronto para ser merged. Como precisei fazer pequenas mudanças no código, criei um PR novo em #290. Vou fechar este, visto que todo código daqui está no outro PR.

@lucasrcezimbra lucasrcezimbra deleted the Lrcezimbra-rs-canoas branch November 2, 2020 19:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
spider Adiciona robô raspador para município(s)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants