Skip to content

Fixes #34 - adding sass #36

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

Closed
wants to merge 1 commit into from
Closed

Fixes #34 - adding sass #36

wants to merge 1 commit into from

Conversation

stefek99
Copy link
Contributor

No description provided.

@stefek99
Copy link
Contributor Author

Apparently commit messages #34 are linked to issues, pull request are not.

But yeah: preboot/angular-webpack#34

@Foxandxss
Copy link
Member

I like the idea but it should be done differently. Like we do at angular2-webpack repo (same organization).

@stefek99
Copy link
Contributor Author

@phra
Copy link
Member

phra commented Jul 13, 2016

yeah, drop node-sass in favor of postcss. @stefek99 are you working on this?

@stefek99
Copy link
Contributor Author

@phra Not really working on this 😈 1000s of other priorities... I had a chat with the mantainer of the repo - he is busy - I'm busy - this guy is busy too - http://www.michaelbromley.co.uk/blog/529/why-i-havent-fixed-your-issue-yet

@phra
Copy link
Member

phra commented Jul 13, 2016

if i have spare time i'll work on this. it's easy, look at USAGE of postcss-loader. ( precss is for using scss syntax with postcss)

@phra
Copy link
Member

phra commented Nov 16, 2016

@stefek99 do you have worked on this? otherwise i have to reject this.

@stefek99
Copy link
Contributor Author

Nope.

@phra
Copy link
Member

phra commented Nov 17, 2016

i'm closing this.

@phra phra closed this Nov 17, 2016
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.

3 participants