search

scrapy / protego

A pure-Python robots.txt parser with support for modern conventions.
BSD 3-Clause "New" or "Revised" License
54 stars 28 forks source link

Add equal-sign to parse safe characters #48

Closed jeroenseegers closed 5 months ago

jeroenseegers commented 5 months ago

This PR fixes the issue mentioned in https://github.com/scrapy/protego/issues/29

codecov[bot] commented 5 months ago

Codecov Report

Merging #48 (37546bf) into master (45e1948) will not change coverage. Report is 1 commits behind head on master. The diff coverage is 100.00%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #48 +/- ## ======================================= Coverage 98.47% 98.47% ======================================= Files 1 1 Lines 328 328 Branches 83 83 ======================================= Hits 323 323 Misses 3 3 Partials 2 2 ``` | [Files](https://app.codecov.io/gh/scrapy/protego/pull/48?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=scrapy) | Coverage Δ | | |---|---|---| | [src/protego.py](https://app.codecov.io/gh/scrapy/protego/pull/48?src=pr&el=tree&filepath=src%2Fprotego.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=scrapy#diff-c3JjL3Byb3RlZ28ucHk=) | `98.47% <100.00%> (ø)` | |
jeroenseegers commented 5 months ago

Thanks!! Would it be possible to create a new release including this change?

Gallaecio commented 5 months ago

Yes, I’ll handle that soon.