Skip to content

Conversation

@maryamariyan
Copy link
Contributor

Makes type safety more strict and accurate for ImplementationDetails

Makes type safety more strict and accurate for ImplementationDetails
@maryamariyan maryamariyan requested a review from a team as a code owner March 20, 2025 17:57
@ghost ghost added Area-IDE untriaged Issues and PRs which have not yet been triaged by a lead labels Mar 20, 2025
@maryamariyan maryamariyan changed the title Update EA contract for Implement with Copilot Update EA contract for Implement with Copilot Mar 20, 2025
@dotnet-policy-service dotnet-policy-service bot added Community The pull request was submitted by a contributor who is not a Microsoft employee. VSCode labels Mar 20, 2025
Won't be possible to fit non-MemberDeclarationSyntax into replacement node
// The .NET Foundation licenses this file to you under the MIT license.
// See the LICENSE file in the project root for more information.

using Microsoft.CodeAnalysis.CSharp.Syntax;
Copy link
Member

Choose a reason for hiding this comment

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

this is not ok. you are referencing C# specific code in a common library. Note: i'm actually surrpised this compiled.

Copy link
Member

@CyrusNajmabadi CyrusNajmabadi left a comment

Choose a reason for hiding this comment

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

Have to change this.

@maryamariyan
Copy link
Contributor Author

closing, this PR is no longer needed

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

Labels

Area-IDE Community The pull request was submitted by a contributor who is not a Microsoft employee. 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.

3 participants