Skip to content

fix: restore recipe parser import - #118

Merged
srpatcha merged 1 commit into
embeddedos-org:masterfrom
vivek3210:fix/recipe-parser-import
Sep 10, 2026
Merged

fix: restore recipe parser import#118
srpatcha merged 1 commit into
embeddedos-org:masterfrom
vivek3210:fix/recipe-parser-import

Conversation

@vivek3210

Copy link
Copy Markdown
Contributor

Summary

  • Define the public parse_recipe function before retaining the _parse_recipe compatibility alias.
  • Add a regression test covering both parser names.

Verification

  • py -m pytest -q tests/ebuild/test_package_recipe.py tests/ebuild/test_smoke_imports.py (64 passed)
  • py -m flake8 ebuild/packages/recipe.py tests/ebuild/test_package_recipe.py (existing line-length and final-newline findings remain)

Copilot AI lite review requested due to automatic review settings September 9, 2026 17:58

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The change cleanly restores the public import path while preserving compatibility, and includes a direct regression test for the aliasing contract.

Pull request overview

This PR restores a public parse_recipe entry point in ebuild.packages.recipe while preserving the historical _parse_recipe name for backward compatibility, and adds a regression test to ensure both names remain available.

Changes:

  • Rename the internal parser function to the public parse_recipe.
  • Keep _parse_recipe as a compatibility alias to the public parser.
  • Add a test asserting _parse_recipe and parse_recipe refer to the same callable.
File summaries
File Description
ebuild/packages/recipe.py Exposes parse_recipe as the primary API while retaining _parse_recipe as a backward-compatible alias.
tests/ebuild/test_package_recipe.py Adds a regression test to ensure the legacy alias still resolves to the public parser.
Review details
  • Files reviewed: 2/2 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@srpatcha
srpatcha merged commit 8b623d5 into embeddedos-org:master Sep 10, 2026
17 of 26 checks passed
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.

3 participants