scrapy / scrapy

Scrapy, a fast high-level web crawling & scraping framework for Python.
https://scrapy.org
BSD 3-Clause "New" or "Revised" License
51.16k stars 10.35k forks source link

Issue #6321: Link extractor all tags and attributes option #6327

Open PJ1256 opened 2 weeks ago

PJ1256 commented 2 weeks ago

This change addresses the first part of issue #6321, allowing the link extractor the option to consider all tags and attributes by passing None in. Since this is my first contribution, and since the docs say to try and break changes into smaller increments, I was planning to wait to address the deny option of the issue until this was successfully addressed. This is my first contribution so please feel free to tell me how I can improve and any problems you see. Thanks!

wRAR commented 2 weeks ago

Please use pre-commit.

Gallaecio commented 1 week ago

The 2 new parameters seem unnecessary.