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

New overrides registry that allows more complex configuration rules for overrides #53

Closed
wants to merge 1 commit into from

Conversation

ivanprado
Copy link
Contributor

The existing overrides registry allowed to configure overrides per domain. But this is not enough in some cases where we need to configure overrides for subdomains or subsections of a site. This PR introduces the class HierarchicalOverridesRegistry that allows configuring these rules in a simple and efficient way. Check the provided documentation for more information.

@codecov
Copy link

codecov bot commented Aug 11, 2021

Codecov Report

Merging #53 (c990523) into master (73af809) will increase coverage by 0.07%.
The diff coverage is 100.00%.

❗ Current head c990523 differs from pull request most recent head 7c7e130. Consider uploading reports for the commit 7c7e130 to get more accurate results

@@            Coverage Diff             @@
##           master      #53      +/-   ##
==========================================
+ Coverage   99.59%   99.66%   +0.07%     
==========================================
  Files           8        8              
  Lines         246      300      +54     
==========================================
+ Hits          245      299      +54     
  Misses          1        1              
Impacted Files Coverage Δ
scrapy_poet/injection.py 100.00% <100.00%> (ø)
scrapy_poet/middleware.py 100.00% <100.00%> (ø)
scrapy_poet/overrides.py 97.91% <100.00%> (+4.58%) ⬆️
scrapy_poet/utils.py 100.00% <100.00%> (ø)

@ivanprado
Copy link
Contributor Author

Close in favour of #56

@ivanprado ivanprado closed this Dec 13, 2021
@BurnzZ BurnzZ deleted the hierarchical_override branch March 17, 2022 07:39
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.

1 participant