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 :topdown selector to match hierarchically #539

Merged
merged 1 commit into from
Sep 10, 2020

Conversation

mtdowling
Copy link
Member

The topdown selector is used to match shapes hierarchically based on a match selector and optional disqualifier selector. This can be used to match shapes that are marked as controlPlane or dataPlane using
inheritance from resource/service bindings, match shapes that use httpBasic auth, etc.

Note: This probably needs more docs, but I wanted to push it up to make sure it looks useful.

@mtdowling mtdowling force-pushed the add-topdown-selector branch from 0808e7b to 661c316 Compare September 9, 2020 05:57
@mtdowling mtdowling requested a review from kstich September 9, 2020 05:57
@mtdowling mtdowling force-pushed the add-topdown-selector branch from 661c316 to 3e21393 Compare September 9, 2020 19:02
@mtdowling mtdowling requested a review from kstich September 9, 2020 19:03
@mtdowling mtdowling force-pushed the add-topdown-selector branch from 3e21393 to 27acf7d Compare September 9, 2020 19:51
The topdown selector is used to match shapes hierarchically based on a
"qualifier" selector and optional "disqualifier" selector. This can be
used to match shapes that are marked as controlPlane or dataPlane using
inheritance from resource/service bindings, match shapes that use
httpBasic auth, etc.
@mtdowling mtdowling force-pushed the add-topdown-selector branch from 27acf7d to 8a624c1 Compare September 9, 2020 19:53
@mtdowling mtdowling merged commit dfe9e77 into master Sep 10, 2020
@mtdowling mtdowling deleted the add-topdown-selector branch September 14, 2020 22:48
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