Skip to content
This repository has been archived by the owner on May 1, 2024. It is now read-only.

Use explicit references so VS for Mac doesn't pick up Design files #2575

Merged
merged 2 commits into from May 2, 2018

Conversation

hartez
Copy link
Contributor

@hartez hartez commented May 1, 2018

Description of Change

We don't want the files in the Design folder to be added as assembly references when installing the nuget package; to avoid this, we make the list of assembly references explicit.

Bugs Fixed

@@ -105,6 +145,17 @@
<file src="..\Xamarin.Forms.Core.Design\bin\$Configuration$\Xamarin.Forms.Core.Design.dll" target="lib\netstandard1.0\Design" />
<file src="..\Xamarin.Forms.Xaml.Design\bin\$Configuration$\Xamarin.Forms.Xaml.Design.dll" target="lib\netstandard1.0\Design" />

<file src="..\Xamarin.Forms.Core.Design\bin\$Configuration$\Xamarin.Forms.Core.Design.dll" target="lib\MonoAndroid10\Design" />
Copy link

Choose a reason for hiding this comment

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

white-space error.

@@ -42,9 +42,49 @@
<dependency id="Tizen.NET" version="4.0.0"/>
</group>
</dependencies>

<references>
Copy link

Choose a reason for hiding this comment

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

Mixing tabs and spaces with the rest of the file.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed.

@samhouts samhouts added this to In Review in v3.0.0 May 1, 2018
@rmarinho rmarinho merged commit 6e95a6d into 3.0.0 May 2, 2018
v3.0.0 automation moved this from In Review to Done May 2, 2018
@hartez hartez deleted the fix-gh2552 branch May 4, 2018 19:03
@samhouts samhouts added this to the 3.1.0 milestone May 8, 2018
@samhouts samhouts removed this from Done in v3.0.0 May 9, 2018
@samhouts samhouts modified the milestones: 3.1.0, 3.0.0 May 14, 2018
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.

None yet

4 participants