Skip to content
This repository has been archived by the owner on Aug 2, 2023. It is now read-only.

Revert dependencies down to release versions of the packages #2573

Merged
merged 5 commits into from
Oct 29, 2018

Conversation

ahsonkhan
Copy link
Member

Adding on top of the changes from #2571.

cc @KrzysztofCwalina, @JeremyKuhne

@ahsonkhan ahsonkhan requested a review from JeremyKuhne October 29, 2018 09:28
<ProjectReference Include="..\System.Text.Utf8String\System.Text.Utf8String.csproj" />
</ItemGroup>
<ItemGroup>
<PackageReference Include="System.IO.Pipelines" Version="$(SystemIOPipelinesVersion)" />
Copy link
Member Author

@ahsonkhan ahsonkhan Oct 29, 2018

Choose a reason for hiding this comment

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

ReaderWriter shouldn't depend on the higher level library (it should be the other way around).

By moving IPipeWritable.cs to Experimental, we can remove this dependency.

@@ -2,16 +2,16 @@
<Import Project="..\..\tools\common.props" />
<PropertyGroup>
<Description>Buffer Readers, Writers, and Transformations</Description>
<TargetFramework>netstandard1.3</TargetFramework>
Copy link
Member Author

Choose a reason for hiding this comment

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

Currently, we can support netstandard1.1.

@ahsonkhan
Copy link
Member Author

@dotnet-bot test Innerloop OSX10.12 Release Build and Test

@ahsonkhan ahsonkhan merged commit fc819d2 into dotnet:master Oct 29, 2018
@ahsonkhan ahsonkhan deleted the UpdateDependenciesToRelease branch October 29, 2018 11:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant