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

big changes #99

Closed
wants to merge 46 commits into from
Closed

big changes #99

wants to merge 46 commits into from

Conversation

searchlyf
Copy link

No description provided.

@searchlyf searchlyf closed this Jul 4, 2019
@searchlyf searchlyf reopened this Jul 4, 2019
@searchlyf searchlyf changed the title z big changes Jul 4, 2019
@liguobao
Copy link

liguobao commented Jul 6, 2019

这里加入了新的代理池吗? 还是咋样的? 我能直接merge这边代码到自己的分支吗?

Copy link

@liguobao liguobao left a comment

Choose a reason for hiding this comment

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

这个提交删除了crawler_booter.py, 但是在run.sh中还有调用.
需要调整一下

@liguobao
Copy link

liguobao commented Jul 6, 2019

合并之后暂时无法直接使用,
Docker构建还会缺sentry_sdk package

@liguobao
Copy link

liguobao commented Jul 6, 2019

而且, 很麻烦的问题在于, 某个站点的校验现在应该怎么处理呢?

@searchlyf
Copy link
Author

没加新的代理池。
crawler_booter.py 改为 run_spiders.py
sentry_sdk 须自己设置
站点的校验只剩下httpbin
改动相当大,merge后会有很多break, 还没写完

@liguobao
Copy link

liguobao commented Jul 7, 2019

站点校验还能支持吗? 我看之前的配置使用run_spiders.py启动不了啦.
我倒是手动merge了很多代码到自己的分支已经解决了部分冲突.
sentry_sdk 在requirements上加上个版本就完事了, 这个倒没什么问题.

@searchlyf
Copy link
Author

searchlyf commented Jul 7, 2019 via email

@liguobao
Copy link

liguobao commented Jul 8, 2019

站点校验应该是需要的, 还有启动方式, 修一下run.sh?

@searchlyf
Copy link
Author

searchlyf@e147fa1 添加你的校验

@searchlyf searchlyf closed this Aug 1, 2019
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.

None yet

2 participants