Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

dotnet-svcutil: optimize the referenced wcf package version and add test. #5519

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

imcarolwang
Copy link
Contributor

@imcarolwang imcarolwang commented Apr 30, 2024

Addresses #5499.

Description:

  1. Correctly resolve netstandard2.0 if the TFM moniker string was passed from VS (WCF CS Tool)
  2. Create clear table mapping between .NET target version and wcf package version, for the various wcf releases which support netstandard2.0, keep the latest version 4.10.* only; update CloseAsync() generation logic to generate it for TFM less than 6.0, and update test baselines
  3. Add only the common wcf references for version 8.0 packages, update corresponding test baselines
  4. Improve wcf assembly reference condition for .NET framework target (use $(TargetFramework.StartsWith('net4') instead of getting the first .NET target and check for equality, this will fix bug when there're multiple .NET framework 4.x targets)

@HongGit
Copy link
Contributor

HongGit commented Sep 20, 2024

/azp run

Copy link

Azure Pipelines successfully started running 2 pipeline(s).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR DO NOT MERGE If for whatever reason you do not want a PR merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants