Skip to content

Conversation

@spike-rabbit
Copy link
Contributor

This is fixing the issue described in #426

@spike-rabbit
Copy link
Contributor Author

@microsoft-github-policy-service agree

@D4N14L
Copy link
Member

D4N14L commented Sep 12, 2024

Looks good to me, but will likely need @octogonz review before merge

@spike-rabbit
Copy link
Contributor Author

spike-rabbit commented Oct 2, 2024

@D4N14L / @octogonz are there any updates?

D4N14L
D4N14L previously approved these changes Oct 2, 2024
@D4N14L
Copy link
Member

D4N14L commented Oct 3, 2024

@octogonz will need to merge the PR. I know he's been quite busy lately but hopefully he sees this.

@spike-rabbit
Copy link
Contributor Author

@octogonz would be really nice to have this merged. Can you please do a review here.

@D4N14L
Copy link
Member

D4N14L commented Nov 21, 2024

@spike-rabbit I've ping'd him on Zulip, which is where he's usually available. Hopefully this will help get some traction here.

@octogonz octogonz merged commit 63b9411 into microsoft:main May 28, 2025
5 checks passed
@octogonz
Copy link
Collaborator

@spike-rabbit FYI we are working on publishing your fix, but there's an issue with the CI pipeline for this repo. Thanks for your patience!

@iclanton

@spike-rabbit spike-rabbit deleted the default-value-inline-content branch May 30, 2025 08:40
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.

3 participants