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

Add CoreWCF Services Support #5473

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

Conversation

afifi-ins
Copy link
Contributor

Rebase latest branch

@afifi-ins afifi-ins force-pushed the AddCoreWcfServicesatest branch from 4e0e6d8 to 6b217de Compare May 9, 2024 19:14
@afifi-ins afifi-ins force-pushed the AddCoreWcfServicesatest branch from 4049c03 to 3f6dad8 Compare June 6, 2024 13:13
@afifi-ins afifi-ins force-pushed the AddCoreWcfServicesatest branch from 6cace94 to 05e132e Compare July 16, 2024 18:06
@afifi-ins afifi-ins force-pushed the AddCoreWcfServicesatest branch 3 times, most recently from 5787e3c to 7eef23d Compare July 23, 2024 04:27
@afifi-ins afifi-ins force-pushed the AddCoreWcfServicesatest branch from c4daf36 to dd2e1d2 Compare August 26, 2024 19:53
@afifi-ins afifi-ins force-pushed the AddCoreWcfServicesatest branch from 0fc1297 to 6d4fa39 Compare September 10, 2024 17:36
@afifi-ins afifi-ins closed this Sep 10, 2024
@afifi-ins afifi-ins reopened this Sep 10, 2024
@afifi-ins afifi-ins force-pushed the AddCoreWcfServicesatest branch from ebc64f8 to bae0e73 Compare September 11, 2024 08:56
@HongGit
Copy link
Contributor

HongGit commented Sep 18, 2024

/azp run

Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@afifi-ins afifi-ins force-pushed the AddCoreWcfServicesatest branch from f092ed7 to 274dc78 Compare September 18, 2024 18:24
@afifi-ins afifi-ins closed this Sep 18, 2024
@afifi-ins afifi-ins reopened this Sep 18, 2024
@HongGit
Copy link
Contributor

HongGit commented Sep 23, 2024

/azp run

Copy link

Azure Pipelines successfully started running 2 pipeline(s).

NuGet.config Outdated
<configuration>
<packageSources>
<clear />
<add key="dotnet-eng" value="https://pkgs.dev.azure.com/dnceng/public/_packaging/dotnet-eng/nuget/v3/index.json" />
<add key="dotnet-tools" value="https://pkgs.dev.azure.com/dnceng/public/_packaging/dotnet-tools/nuget/v3/index.json" />
<add key="dotnet-public" value="https://pkgs.dev.azure.com/dnceng/public/_packaging/dotnet-public/nuget/v3/index.json" />
<add key="nuget" value="https://api.nuget.org/v3/index.json" />
Copy link
Contributor

Choose a reason for hiding this comment

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

If I remember this correctly, this is not allowed, We should achieve this through dotnet-public.

NuGet.config Outdated
<configuration>
<packageSources>
<clear />
<add key="dotnet-eng" value="https://pkgs.dev.azure.com/dnceng/public/_packaging/dotnet-eng/nuget/v3/index.json" />
<add key="dotnet-tools" value="https://pkgs.dev.azure.com/dnceng/public/_packaging/dotnet-tools/nuget/v3/index.json" />
<add key="dotnet-public" value="https://pkgs.dev.azure.com/dnceng/public/_packaging/dotnet-public/nuget/v3/index.json" />
<add key="nuget" value="https://api.nuget.org/v3/index.json" />
<add key="CoreWCF_Dev" value="https://pkgs.dev.azure.com/dotnet/CoreWCF/_packaging/CoreWCF/nuget/v3/index.json" />
Copy link
Contributor

Choose a reason for hiding this comment

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

Couldn't we just use the latest CoreWCF package for this? Or perhaps this should be added to the dotnet-public?


<Target Name="InstallDotNet">
<ItemGroup>
<AdditionalDotNetPackage Include="8.0.5">
Copy link
Contributor

Choose a reason for hiding this comment

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

This seems to be downgrade; can we use what is available provided by arcade?

@afifi-ins afifi-ins force-pushed the AddCoreWcfServicesatest branch from d2b48bc to 5301ab2 Compare October 29, 2024 17:08
@afifi-ins afifi-ins force-pushed the AddCoreWcfServicesatest branch from 3370a51 to a5b1c1c Compare November 11, 2024 14:50
@afifi-ins
Copy link
Contributor Author

/azp run

Copy link

Commenter does not have sufficient privileges for PR 5473 in repo dotnet/wcf

@afifi-ins afifi-ins closed this Nov 12, 2024
@afifi-ins afifi-ins reopened this Nov 12, 2024
@imcarolwang
Copy link
Contributor

/azp run

Copy link

Pull request contains merge conflicts.

@afifi-ins afifi-ins force-pushed the AddCoreWcfServicesatest branch from b9c1979 to 0f85880 Compare November 15, 2024 04:45
@afifi-ins afifi-ins force-pushed the AddCoreWcfServicesatest branch from f9dc9da to d1ec63f Compare November 21, 2024 20:11
@afifi-ins afifi-ins closed this Dec 4, 2024
@afifi-ins afifi-ins reopened this Dec 4, 2024
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.

3 participants