Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

Use fallible callbacks for Mapping placeholders #1217

Merged
merged 2 commits into from
Aug 31, 2021

Conversation

ranweiler
Copy link
Member

The template expansion machinery allows Mapping placeholders which use a partial callback to provide an expansion value on demand. Add fallibility to the callback signature, which lets us avoid distinguish between callback errors and absent values. Update existing Mapping callbacks to return errors in some places where we currently map them to None.

@bmc-msft bmc-msft merged commit 883518e into microsoft:main Aug 31, 2021
@ranweiler ranweiler deleted the expand-err branch August 31, 2021 17:45
@ghost ghost locked as resolved and limited conversation to collaborators Sep 30, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants