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

Merge bundle interfaces #2685

Open
Tracked by #2678
carolynvs opened this issue Mar 31, 2023 · 0 comments
Open
Tracked by #2678

Merge bundle interfaces #2685

carolynvs opened this issue Mar 31, 2023 · 0 comments
Labels
pep003-advanced-dependencies Implementation of the Advanced Dependencies proposal

Comments

@carolynvs
Copy link
Member

carolynvs commented Mar 31, 2023

When a dependency is defined there are two interfaces involved:

  • base interface, defined by how the dependency is used and wired up to the parent bundle.
  • declared interface, defined by the bundle interface field on the dependency.

Porter needs to merge the two before evaluating the bundle interface. The base interface is very sparse and mostly involves determining if the bundle should have a particular output, param, or cred defined. The defined interface can be more specific and include json schema and interface ids. When we combine the two, the declared interface should be merged "on top of" the base interface.

ℹ️ Read PEP003 - Advanced Dependencies for context about how dependencies should work, design details, and notes about desired behavior.

@carolynvs carolynvs added the pep003-advanced-dependencies Implementation of the Advanced Dependencies proposal label Mar 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pep003-advanced-dependencies Implementation of the Advanced Dependencies proposal
Projects
No open projects
Status: No status
Development

No branches or pull requests

1 participant