Skip to content
This repository was archived by the owner on Oct 24, 2025. It is now read-only.

Set TargetPath to avoid MSB8012 warning from Visual Studio#2013

Open
saper wants to merge 1 commit into
sass:masterfrom
saper:msb8012
Open

Set TargetPath to avoid MSB8012 warning from Visual Studio#2013
saper wants to merge 1 commit into
sass:masterfrom
saper:msb8012

Conversation

@saper

@saper saper commented Apr 18, 2016

Copy link
Copy Markdown
Member

Well, actually it seems fine. @am11 would be cool if you could have a look at this!

@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 78.844% when pulling 8323d7b on saper:msb8012 into dad3022 on sass:master.

@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage decreased (-0.6%) to 78.288% when pulling 8323d7b on saper:msb8012 into dad3022 on sass:master.

@am11

am11 commented Apr 19, 2016

Copy link
Copy Markdown
Contributor

Does renaming TargetPath to SasscPath fix the warning? (i.e. without defining extra variables)

@saper

saper commented Apr 19, 2016

Copy link
Copy Markdown
Member Author

It might, actually my first thought was that this is just a coincidence...

@xzyfer

xzyfer commented May 2, 2016

Copy link
Copy Markdown
Contributor

@saper any action to be taken here? Please ship it or close it :)

@xzyfer xzyfer added the Build label May 20, 2016
@xzyfer

xzyfer commented Feb 19, 2017

Copy link
Copy Markdown
Contributor

VS build files have seen a lot of changes recently. Is this still relevant?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants