Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Added synchronous support #39

Closed
wants to merge 1 commit into from
Closed

Added synchronous support #39

wants to merge 1 commit into from

Conversation

aputinski
Copy link

I ran into several issues where asynchronous processing was causing my final output to be ordered based on how long each file took to process imports.

This PR simply allows a ?sync parameter to be passed in.

@jhnns
Copy link
Member

jhnns commented Jan 9, 2015

Cool, thx! 👍 Could you also add some tests?

jhnns added a commit that referenced this pull request Mar 22, 2015
@jhnns
Introduce CHANGELOG
866f386 
- Moved `node-sass^3.0.0-alpha.0` to `peerDependencies` [#28](#28)
- Using webpack's module resolver as custom importer [#39](#31)
- Add synchronous compilation support for usage with [enhanced-require](https://github.com/webpack/enhanced-require) [#39](#39)
@jhnns jhnns mentioned this pull request Mar 22, 2015
Merged
@jhnns
Copy link
Member

jhnns commented Mar 22, 2015

Synchronous compilation has been implemented, but there is no flag yet...

@jhnns jhnns mentioned this pull request Mar 22, 2015
Closed
@jhnns jhnns closed this Mar 22, 2015
@confiks confiks mentioned this pull request Mar 24, 2015
Closed
@Hufschmidt Hufschmidt mentioned this pull request Jun 27, 2016
Closed
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@aputinski @jhnns