Skip to content

Add fluent style to NullConditionalExpression#3800

Merged
m-nash merged 14 commits intomicrosoft:mainfrom
pshao25:expression3798
Jul 18, 2024
Merged

Add fluent style to NullConditionalExpression#3800
m-nash merged 14 commits intomicrosoft:mainfrom
pshao25:expression3798

Conversation

@pshao25
Copy link
Copy Markdown
Contributor

@pshao25 pshao25 commented Jul 10, 2024

Fix #3798, Fix #3829

@microsoft-github-policy-service microsoft-github-policy-service Bot added the emitter:client:csharp Issue for the C# client emitter: @typespec/http-client-csharp label Jul 10, 2024
@azure-sdk
Copy link
Copy Markdown
Collaborator

API change check

APIView has identified API level changes in this PR and created following API reviews.

Microsoft.Generator.CSharp

@pshao25 pshao25 changed the title Add fluent style to NullConditionalEXpression Add fluent style to NullConditionalExpression Jul 11, 2024
@pshao25 pshao25 marked this pull request as ready for review July 11, 2024 05:36
@azure-sdk
Copy link
Copy Markdown
Collaborator

azure-sdk commented Jul 12, 2024

API change check

APIView has identified API level changes in this PR and created following API reviews.

Microsoft.Generator.CSharp
Microsoft.Generator.CSharp.ClientModel

@m-nash m-nash enabled auto-merge July 18, 2024 16:25
@m-nash m-nash added this pull request to the merge queue Jul 18, 2024
Merged via the queue into microsoft:main with commit ac44dc3 Jul 18, 2024
@pshao25 pshao25 deleted the expression3798 branch July 19, 2024 01:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

emitter:client:csharp Issue for the C# client emitter: @typespec/http-client-csharp

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove the nullConditional flag in ValueExpression.Property NullConditionalExpression should also be used in a fluent style

6 participants