Skip to content

Conversation

@michaeljin89757
Copy link

Add a new if statement to check for isnew after the constraints are checked.

…hecked.

Add a new if statement to check for isnew after the constraints are checked.
@anmeng10101
Copy link
Collaborator

@michaeljin89757 please check the integration error and fix it.

@hiihellox10
Copy link
Contributor

You should write a test case to reproduce the issue and fix it.

@michaeljin89757 michaeljin89757 changed the title Add a new if statement to check for isnew after the constraints are c… [.NET] Compound Type Constraint Does Not Render Apr 13, 2021
Michael Jin added 2 commits April 14, 2021 13:43
2. Append new condition to add comma when there is constraint.
@dnfadmin
Copy link

dnfadmin commented Apr 14, 2021

CLA assistant check
All CLA requirements met.

Copy link
Member

@joelmartinez joelmartinez left a comment

Choose a reason for hiding this comment

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

Thanks for this contribution. There's a few items of feedback ... and then a small reminder that when you add the new test method, you'll have to again change the expected XMLs to reflect this new item.

@joelmartinez joelmartinez merged commit decaa24 into mono:develop Apr 26, 2021
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.

5 participants