Skip to content

feat: add Emily Bites scraper (emilybites.com)#1971

Open
flaminiom wants to merge 1 commit into
hhursev:mainfrom
flaminiom:add-emilybites
Open

feat: add Emily Bites scraper (emilybites.com)#1971
flaminiom wants to merge 1 commit into
hhursev:mainfrom
flaminiom:add-emilybites

Conversation

@flaminiom

Copy link
Copy Markdown

Summary

Emily Bites (emilybites.com) is a popular lightened-up comfort food blog that uses the WP Recipe Maker plugin. This PR adds a minimal scraper following the established WPRMMixin + AbstractScraper pattern used by 100+ other sites in this library.

Changes:

  • Add recipe_scrapers/emilybites.py
  • Register EmilyBites in __init__.py (alphabetically between EggsCa and ElaVegan)
  • Add test data: tests/test_data/emilybites.com/emilybites_1.testhtml + emilybites_1.json

All 1098 tests pass.

🤖 Generated with Claude Code

Site uses WP Recipe Maker plugin, so WPRMMixin provides all method
implementations. Includes test data from a Creamy Tomato Pasta recipe.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

@jknndy jknndy left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @flaminiom , thanks for the contribution. The wprm mixin is only used in instances where equipment is available, as it is not a schema field it has its only implementation with the mixin. I've placed two comments to remove it, once done this will be good to go!

from ._wprm import WPRMMixin


class EmilyBites(WPRMMixin, AbstractScraper):

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
class EmilyBites(WPRMMixin, AbstractScraper):
class EmilyBites(AbstractScraper):

@@ -0,0 +1,8 @@
from ._abstract import AbstractScraper
from ._wprm import WPRMMixin

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
from ._wprm import WPRMMixin

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.

2 participants