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.

Sign up for GitHub

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

Jump to bottom

Fix flatten signature #9117

Merged
merged 4 commits into from
Nov 28, 2022
Merged

Fix flatten signature #9117

merged 4 commits into from
Nov 28, 2022

Conversation

jeskew
Copy link
Contributor

@jeskew jeskew commented Nov 27, 2022 •
edited by ghost
Loading

Resolves #9116.

Encountered this while working on an unrelated task which required calculating the return type of flatten, so that work is also included here.

Microsoft Reviewers: Open in CodeFlow

@jeskew jeskew requested a review from a team November 27, 2022 21:42
majastrz
majastrz approved these changes Nov 28, 2022
View reviewed changes
Copy link
Member

@majastrz majastrz left a comment

Choose a reason for hiding this comment

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

:shipit:

@jeskew jeskew enabled auto-merge (squash) November 28, 2022 19:51
@jeskew jeskew merged commit e94b6c7 into main Nov 28, 2022
@jeskew jeskew deleted the jeskew/flatten-fixup branch November 28, 2022 20:20
@anthony-c-martin
Copy link
Member

Thanks for fixing this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@majastrz majastrz majastrz approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

flatten definition in Bicep does not match definition in runtime
3 participants
@jeskew @anthony-c-martin @majastrz