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

Windows: Add editorconfig and fix filterpaths for latest libsass #184

Merged
merged 1 commit into from
Sep 5, 2016

Conversation

nschonni
Copy link
Contributor

@nschonni nschonni commented Sep 5, 2016

  • Add missing LIBSASS_INCLUDES_DIR reference
  • Fix AdditionalIncludeDirectories so base.h and the other include\sass
    files are found
  • Update filter paths from latest libsass.targets

- Add missing LIBSASS_INCLUDES_DIR property
- Fix AdditionalIncludeDirectories so base.h and the other include\sass
files are found
- Update filter paths from latest libsass.targets
- Changes LIBSASS_DIR to a peer folder instead of a parent directory
@xzyfer
Copy link
Contributor

xzyfer commented Sep 5, 2016

Whoa some of that stuff was ancient

@nschonni
Copy link
Contributor Author

nschonni commented Sep 5, 2016

Yeah, I'll take a look at setting up Appveyor later, but I thought I'd get it building again separately

@xzyfer
Copy link
Contributor

xzyfer commented Sep 5, 2016

Do you have commit privileges? If so 🚢

@nschonni
Copy link
Contributor Author

nschonni commented Sep 5, 2016

Nope, just node-sass, but I usually try to avoid self-merging my stuff. It's shared blame when stuff blows up then 😜

@xzyfer
Copy link
Contributor

xzyfer commented Sep 5, 2016

Build stuff is mostly gibberish to me, but I'll peek my head into any PR emails when I have a moment.
I've invited you to the @sass/sassc so that you're not blocked me :)

@xzyfer
Copy link
Contributor

xzyfer commented Sep 5, 2016

If you're in the area, getting CI setup would be 👌 for lowering risk of self-merging.

@nschonni nschonni merged commit 731ca5c into sass:master Sep 5, 2016
@nschonni nschonni deleted the editorconfig branch September 5, 2016 21:35
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.

2 participants