Skip to content

Conversation

@AlekseyTs
Copy link
Contributor

No description provided.

@AlekseyTs
Copy link
Contributor Author

@CyrusNajmabadi Please review

class C : {|CS0535:ITest|}
{
}
namespace System.Runtime.CompilerServices
Copy link
Member

@CyrusNajmabadi CyrusNajmabadi Apr 17, 2025

Choose a reason for hiding this comment

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

given the test duplication, my preference is this is pulled into a constant at the start of the file and included in the tests that need it. if you're on a time crunch, this isn't blocking, and i can do that as a followup. But if it's simple (and/or copilot can help :)) then that would be appreciated. #Resolved

Copy link
Contributor Author

Choose a reason for hiding this comment

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

my preference is this is pulled into a constant at the start of the file and included in the tests that need it.

Will do

@AlekseyTs
Copy link
Contributor Author

@CyrusNajmabadi Are we good to merge with a single sign-off?

@CyrusNajmabadi
Copy link
Member

Yup!

@AlekseyTs AlekseyTs merged commit 6c8b098 into dotnet:features/UserDefinedCompoundAssignment Apr 18, 2025
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area-IDE Feature-UserDefinedCompoundAssignmentOperators untriaged Issues and PRs which have not yet been triaged by a lead VSCode

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants