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

CB-11825 Windows dll file won't be copied as resource while adding cu… #199

Merged
merged 1 commit into from Oct 12, 2016
Merged

CB-11825 Windows dll file won't be copied as resource while adding cu… #199

merged 1 commit into from Oct 12, 2016

Conversation

daserge
Copy link
Contributor

@daserge daserge commented Oct 5, 2016

…stom plugin to a UWP project

Platforms affected

Windows

What does this PR do?

Changing Package action from Content to None because Content does not work for Mobile - dlls were not being packaged into appx

What testing has been done on this change?

Manual and auto tests

Checklist

  • Reported an issue in the JIRA database
  • Commit message follows the format: "CB-3232: (android) Fix bug with resolving file paths", where CB-xxxx is the JIRA ID & "android" is the platform affected.
  • Added automated test coverage as appropriate for this change.

@codecov-io
Copy link

codecov-io commented Oct 5, 2016

Current coverage is 74.34% (diff: 100%)

Merging #199 into master will not change coverage

@@             master       #199   diff @@
==========================================
  Files            14         14          
  Lines          1949       1949          
  Methods         366        366          
  Messages          0          0          
  Branches        392        392          
==========================================
  Hits           1449       1449          
  Misses          500        500          
  Partials          0          0          

Powered by Codecov. Last update 5073029...1417b93

…stom plugin to a UWP project

Changing Package action from Content to None because Content does not work for Mobile - dlls were not being packaged into appx
@asfgit asfgit merged commit 1417b93 into apache:master Oct 12, 2016
@daserge daserge deleted the CB-11825 branch October 12, 2016 11:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants