Skip to content

Binplacing the xml file#6886

Merged
Anipik merged 1 commit intodotnet:masterfrom
Anipik:docFile
Jan 30, 2021
Merged

Binplacing the xml file#6886
Anipik merged 1 commit intodotnet:masterfrom
Anipik:docFile

Conversation

@Anipik
Copy link
Copy Markdown
Contributor

@Anipik Anipik commented Jan 30, 2021

We were missing the binplacing the xml file in certain cases. This fixes that

Copy link
Copy Markdown
Contributor

@dougbu dougbu left a comment

Choose a reason for hiding this comment

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

Makes sense

@Anipik Anipik merged commit ec1c8cf into dotnet:master Jan 30, 2021
@Anipik Anipik deleted the docFile branch January 30, 2021 22:23
<ItemGroup>
<BinPlaceItem Condition="Exists('$(TargetPath)')" Include="$(TargetPath)" />
<BinPlaceItem Condition="Exists('$(TargetDir)$(TargetName).pdb')" Include="$(TargetDir)$(TargetName).pdb" />
<BinPlaceItem Condition="Exists('$(TargetDir)$(TargetName).xml')" Include="$(TargetDir)$(TargetName).xml" />
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

isn't the xml file not part of BinPlaceCopyToOutputDirectoryItems?

akoeplinger pushed a commit to akoeplinger/arcade that referenced this pull request Apr 12, 2021
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.

4 participants