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.

Sign up for GitHub

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

Jump to bottom

Johluo/remove migrated files #3160

Merged
merged 5 commits into from
Apr 7, 2020
Merged

Johluo/remove migrated files #3160

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
212d133
Remove migrated packages
Apr 7, 2020
af363c6
Remove reference assemblies and associated logic
Apr 7, 2020
10d2ec4
Cleanup slns and projects
Apr 7, 2020
3a99a49
Remove sample
Apr 7, 2020
ffc9824
Fix codecheck
Apr 7, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
Remove migrated packages
  • Loading branch information
John Luo committed Apr 7, 2020
commit 212d133337888738ec0a1cd36771c580a52dc5c0
2 changes: 0 additions & 2 deletions .github/CODEOWNERS
View file
Open in desktop
Validating CODEOWNERS rules …
Original file line number Diff line number Diff line change
Expand Up @@ -10,5 +10,3 @@
/src/Caching/ @tratcher @anurse
Copy link
Author

Choose a reason for hiding this comment

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

we should probably hand these off too @dotnet/extensions-migration .

Copy link

Choose a reason for hiding this comment

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

@JunTaoLuo are you able to provide details as to where the source for logging & logging abstractions now live? as the master branch no longer contains the source as far as I can tell?

Copy link
Member

Choose a reason for hiding this comment

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

https://github.com/dotnet/runtime/blob/6ef538d46fc971f351a22cbe854717f4ada59d89/src/libraries/Microsoft.Extensions.Logging.Abstractions/src/ILogger.cs

Copy link

Choose a reason for hiding this comment

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

See aspnet/Announcements#411 for the full list of packages and where the code lives after this migration is complete.

Copy link

Choose a reason for hiding this comment

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

@Tratcher @anurse thanks for pointing me to the announcement and new location for the source; it may be good to also update the README.md accordingly with a more information and/or to redirect others to the new location for source; as people may still think the namespace/package relates to this repo and not the runtime repo if they have missed the announcement.

Copy link
Author

Choose a reason for hiding this comment

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

@bradygaster how far along are you with updating the readme?

Copy link

Choose a reason for hiding this comment

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

https://github.com/aspnet/Logging still points to https://github.com/dotnet/extensions which is quite confusing now

Copy link
Member

Choose a reason for hiding this comment

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

@poizan42 did you see the updated ReadMe in extensions?

Copy link

Choose a reason for hiding this comment

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

Yes, but the description at https://github.com/aspnet/Logging should probably still be updated to point to https://github.com/dotnet/runtime/tree/master/src/libraries/Microsoft.Extensions.Logging or something

/src/Configuration/ @tratcher @anurse
/src/Hosting/ @tratcher @anurse
/src/Primitives/ @tratcher @anurse
/src/Shared/src/HostFactoryResolver/ @tratcher @anurse
2 changes: 1 addition & 1 deletion Directory.Build.props
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@
<ItemGroup Condition=" '$(IsUnitTestProject)' == 'true' ">
<ProjectReference Include="$(MSBuildThisFileDirectory)src\TestingUtils\Microsoft.AspNetCore.Testing\src\Microsoft.AspNetCore.Testing.csproj" />
<Reference Include="Moq" />
<!-- Bring in the latest NETStandard.Library to avoid too many package references from
<!-- Bring in the latest NETStandard.Library to avoid too many package references from
Moq & xunit's dependencies -->
<Reference Include="NETStandard.Library" />
</ItemGroup>
Expand Down
3 changes: 0 additions & 3 deletions eng/Build.props
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,8 +1,5 @@
<Project>
<ItemGroup>
<ProjectToBuild Include="$(RepoRoot)src\**\*.csproj" Condition="'$(BuildCSharp)' != 'false'"/>
<ProjectToBuild Include="$(RepoRoot)src\**\*.npmproj" Condition="'$(BuildNodeJS)' == 'true'">
<RestoreInParallel>false</RestoreInParallel>
</ProjectToBuild>
</ItemGroup>
</Project>
11 changes: 0 additions & 11 deletions eng/repo.targets
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -53,15 +53,4 @@
SkipNonexistentProjects="true" />
</Target>

<Target Name="EnsureFunctionalTestLogsPreserved">
<PropertyGroup>
<FunctionalLogDir>$(ArtifactsDir)log\Microsoft.Extensions.Logging.Testing.Tests\$(DefaultNetCoreTargetFramework)\AssemblyTestLogTests\</FunctionalLogDir>
<FlakyTestLog>$(FunctionalLogDir)FunctionalLogs_LogsPreservedFromFlakyRun.log</FlakyTestLog>
<NonFlakyTestLog>$(FunctionalLogDir)FunctionalLogs_LogsPreservedFromNonFlakyRun.log</NonFlakyTestLog>
</PropertyGroup>
<Error Condition="!Exists('$(FlakyTestLog)')" Text="Flaky test functional logs not found." />
<Error Condition="!Exists('$(FlakyTestLog)')" Text="Non flaky test functional logs not found." />

<Message Text="All functional test logs preserved" Importance="High" />
</Target>
</Project>
0 ...Common/ConfigurationProviderExtensions.cs → ...n/test/ConfigurationProviderExtensions.cs
View file
Open in desktop
File renamed without changes.
0 ...fig/test/ConfigurationProviderTestBase.cs → ...son/test/ConfigurationProviderTestBase.cs
View file
Open in desktop
File renamed without changes.
12 changes: 0 additions & 12 deletions ...Config.NewtonsoftJson/test/Microsoft.Extensions.Configuration.NewtonsoftJson.Tests.csproj
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,18 +4,6 @@
<TargetFrameworks>$(DefaultNetCoreTargetFramework);net472</TargetFrameworks>
</PropertyGroup>

<ItemGroup>
<Compile Include="..\..\Config\test\ConfigurationProviderTestBase.cs">
<Link>test\ConfigurationProviderTestBase.cs</Link>
</Compile>
<Compile Include="..\..\Config\testassets\Test.Common\ConfigurationProviderExtensions.cs">
<Link>testassets\Common\ConfigurationProviderExtensions.cs</Link>
</Compile>
<Compile Include="..\..\Config\testassets\Test.Common\TestStreamHelpers.cs">
<Link>testassets\Common\TestStreamHelpers.cs</Link>
</Compile>
</ItemGroup>

<ItemGroup>
<Reference Include="Microsoft.Extensions.Configuration.Abstractions" />
<Reference Include="Microsoft.Extensions.Configuration.Binder" />
Expand Down
0 ...stassets/Test.Common/TestStreamHelpers.cs → ....NewtonsoftJson/test/TestStreamHelpers.cs
View file
Open in desktop
File renamed without changes.
20 changes: 0 additions & 20 deletions src/DependencyInjection/DI.External.Tests/test/Autofac.cs
View file
Open in desktop

This file was deleted.

20 changes: 0 additions & 20 deletions src/DependencyInjection/DI.External.Tests/test/DryIoc.cs
View file
Open in desktop

This file was deleted.

24 changes: 0 additions & 24 deletions src/DependencyInjection/DI.External.Tests/test/Grace.cs
View file
Open in desktop

This file was deleted.

22 changes: 0 additions & 22 deletions src/DependencyInjection/DI.External.Tests/test/Lamar.cs
View file
Open in desktop

This file was deleted.

21 changes: 0 additions & 21 deletions src/DependencyInjection/DI.External.Tests/test/LightInject.cs
View file
Open in desktop

This file was deleted.

30 changes: 0 additions & 30 deletions ...ernal.Tests/test/Microsoft.Extensions.DependencyInjection.ExternalContainers.Tests.csproj
View file
Open in desktop

This file was deleted.

32 changes: 0 additions & 32 deletions ...endencyInjection/DI.External.Tests/test/SkippableDependencyInjectionSpecificationTests.cs
View file
Open in desktop

This file was deleted.

16 changes: 0 additions & 16 deletions src/DependencyInjection/DI.External.Tests/test/StashBox.cs
View file
Open in desktop

This file was deleted.

29 changes: 0 additions & 29 deletions src/DependencyInjection/DI.External.Tests/test/StructureMap.cs
View file
Open in desktop

This file was deleted.

21 changes: 0 additions & 21 deletions src/DependencyInjection/DI.External.Tests/test/Unity.cs
View file
Open in desktop

This file was deleted.

Loading