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

Support System.ComponentModel.Composition (MEFv1) on nuget.org #72

Merged
merged 3 commits into from
Mar 16, 2018

Conversation

AArnott
Copy link
Member

@AArnott AArnott commented Mar 13, 2018

This updates our new Microsoft.VisualStudio.Composition.NetFxAttributes package and VS MEF engine to support the MEFv1 attributes as defines for .NETStandard 2.0 now found on NuGet.org.

Closes #66

With the System.ComponentModel.Composition NuGet package, the MEFv1 engine runs on netcoreapp2.0 and all the MEF attributes are available on .NETStandard2.0, so we don't need to define them any more. Instead, we revert to type forwarders and have a transitive package dependency on the official package.
@codecov-io
Copy link

Codecov Report

Merging #72 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #72   +/-   ##
=======================================
  Coverage   85.86%   85.86%           
=======================================
  Files          67       67           
  Lines        5581     5581           
  Branches      865      865           
=======================================
  Hits         4792     4792           
  Misses        552      552           
  Partials      237      237
Impacted Files Coverage Δ
...position/Configuration/SerializationContextBase.cs 90.87% <ø> (ø) ⬆️
...lStudio.Composition/Configuration/NetFxAdapters.cs 76.56% <ø> (ø) ⬆️
...icrosoft.VisualStudio.Composition/PartDiscovery.cs 84.1% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5f8f377...8186263. Read the comment docs.

@AArnott AArnott merged commit 0473004 into master Mar 16, 2018
@terrajobst
Copy link
Member

Makes sense to me

@AArnott AArnott deleted the dev/andarno/WindowsCompatMEF branch March 16, 2018 02:09
AArnott pushed a commit that referenced this pull request Jun 7, 2022
Bumps [Microsoft.NET.Test.Sdk](https://github.com/microsoft/vstest) from 16.7.0 to 16.7.1.
- [Release notes](https://github.com/microsoft/vstest/releases)
- [Commits](microsoft/vstest@v16.7.0...v16.7.1)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants