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
Merge Slot classes back? #4296
Comments
This is even more confusing, actually. And it's not what I wanted to create this ticket about initially. There is another There is also |
If nothing else, we should rename those classes 😄 |
Yeah, they should definitely be called Just get rid of them, if they have no point, aye? |
Hi can I work on this? |
hi @apurvTri123 , this is a discussion ticket, we are not sure what do we want to do with this, if anything at all. |
In 409aaad
scrapy.core.engine.Slot
was extracted fromscrapy.core.engine.ExecutionEngine
and they had a many-to-one rel. In a84e5f8 that rel was changed to one-to-one, and I'm not sure what does this split achieve now.The text was updated successfully, but these errors were encountered: