fix: CodeBlockParser has been renamed to PythonCodeBlockParser

[azzamsa]

fixes #5298

[wRAR]

This will break with old Sybil.

[codecov]

Codecov Report

Merging #5299 (ecf1482) into master (3225de7) will decrease coverage by 0.16%.
The diff coverage is n/a.

❗ Current head ecf1482 differs from pull request most recent head 67994d1. Consider uploading reports for the commit 67994d1 to get more accurate results

@@            Coverage Diff             @@
##           master    #5299      +/-   ##
==========================================
- Coverage   88.53%   88.36%   -0.17%     
==========================================
  Files         163      163              
  Lines       10616    10616              
  Branches     1558     1558              
==========================================
- Hits         9399     9381      -18     
- Misses        942      960      +18     
  Partials      275      275              

Impacted Files	Coverage Δ
scrapy/robotstxt.py	75.30% <0.00%> (-22.23%)	⬇️

[azzamsa]

I have updated the code to handle old Sybil version.
Taken from @Gallaecio repo

[Gallaecio]

Shall we pint to >=3,<4, and just switch the code to the new way? (it seems to follow semantic versioning, so we should be safe with that range)

[Gallaecio]

I’m OK with this fix, but I also think that we don’t need backward compatibility for a test-only dependency like this. I would just make the change for the 3.0 API, and possibly cap to >=3,<4.

[azzamsa]

Any update about this PR?
Thanks.