Skip to content

state/plan: Add output type and relevant_attributes#53

Merged
radeksimko merged 3 commits intomainfrom
f-output-type
May 25, 2022
Merged

state/plan: Add output type and relevant_attributes#53
radeksimko merged 3 commits intomainfrom
f-output-type

Conversation

@radeksimko
Copy link
Copy Markdown
Member

@radeksimko radeksimko commented May 9, 2022

Closes #52

I initially started with just output type, but ended up implementing relevant_attributes because the existing test configuration I copied from 110_basic produces relevant_attributes, so instead of changing that configuration I thought it's best to just account for it 🤷🏻‍♂️


Related to:

@radeksimko radeksimko requested review from alisdair and jbardin May 9, 2022 12:17
Copy link
Copy Markdown
Member

@cam-stitt cam-stitt left a comment

Choose a reason for hiding this comment

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

LGTM!

@radeksimko radeksimko merged commit ea4b962 into main May 25, 2022
@radeksimko radeksimko deleted the f-output-type branch May 25, 2022 07:09
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.

Reflect output type

2 participants