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

fix: Android DesignTimeBuild issue with MsBuild.Sdk.Extras #390

Merged
merged 2 commits into from
Aug 17, 2019

Conversation

RLittlesII
Copy link
Member

What kind of change does this PR introduce?
Working around an Android build issue with MsBuild.Sdk.Extras

What is the current behavior?
Splat doesn't build.

What is the new behavior?
Splat builds.

What might this PR break?
Nothing that isn't already broken.

@RLittlesII RLittlesII requested a review from a team August 17, 2019 00:48
@codecov
Copy link

codecov bot commented Aug 17, 2019

Codecov Report

Merging #390 into master will decrease coverage by 0.08%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #390      +/-   ##
==========================================
- Coverage   71.16%   71.08%   -0.09%     
==========================================
  Files          66       66              
  Lines        3676     3676              
  Branches      335      335              
==========================================
- Hits         2616     2613       -3     
- Misses       1015     1017       +2     
- Partials       45       46       +1
Impacted Files Coverage Δ
src/Splat.NLog/NLogLogger.cs 73.74% <0%> (-0.84%) ⬇️

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 9fa5f68...d5265e5. Read the comment docs.

@glennawatson glennawatson merged commit c2c0fc8 into master Aug 17, 2019
@glennawatson glennawatson deleted the fix-build branch August 17, 2019 01:05
@lock lock bot locked and limited conversation to collaborators Nov 15, 2019
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.

2 participants