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

Add support and tests for capability specification via provides #80

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

pipex
Copy link
Contributor

@pipex pipex commented Dec 30, 2024

This updates the satisfiesChildContract and getNotSatisfiedRequirements functions to check for capabilities defined via provides on the current context. This PR is just surfacing some functionality that was already partially there but never tested.

Change-type: minor
Depends-on: #79

@pipex pipex marked this pull request as draft December 30, 2024 14:05
@pipex pipex force-pushed the capability-support branch 10 times, most recently from eeb6ea9 to 1c1c1a3 Compare January 7, 2025 21:40
@pipex pipex force-pushed the capability-support branch 2 times, most recently from d2b2177 to db65841 Compare January 15, 2025 15:38
This removes some code duplication between `satisfiesChildContract` and
`getNotSatisfiedRequirements`

Change-type: patch
@pipex pipex force-pushed the capability-support branch from db65841 to 2180cbb Compare January 15, 2025 16:17
This lines up better with the original specification and how contracts
are being used by the OS

Change-type: minor
@pipex pipex force-pushed the capability-support branch from 2180cbb to 978c919 Compare January 15, 2025 16:34
@pipex pipex marked this pull request as ready for review January 15, 2025 16:34
@pipex pipex requested a review from a team January 15, 2025 16:34
@flowzone-app flowzone-app bot enabled auto-merge January 15, 2025 16:37
Using array cardinalities would result in the blueprint using the
`Array.filter` function as the filter for the selector, causing the
filtered set to always have size 0

Change-type: patch
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