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

no caching when in development mode #776

Closed
wants to merge 1 commit into from

Conversation

jakob-stoeck
Copy link
Contributor

Hi, when using watch with a file which has @import statements, the cache prevented reloading of those imported files. I think you can assume that when in development env and using watch that you really want to reload the file.

@timhettler
Copy link

Would love to see this implemented!

@paulspringett
Copy link

+1

@gboudreau
Copy link

@agatronic Do you know why this was never pulled into this repo? Did it just fell in a crack, or is it missing something to be considered? Considering the number of watchers of #47, I think a lot of people would benefit from this, if it indeed resolves that issue as @DomBlack say.
I'm pretty sure many developers would be happy to modify this as required for this to get merged into the @cloudhead repo.
Thanks.

@lukeapage
Copy link
Member

@gboudreau its not usable as a pull request and all browser bugs are low priority, thats why... but.. if the if statement had brackets and was less.env != 'development' then I see no reason why to not accept it.

@gboudreau
Copy link

Thanks. I submitted pull request 995 that includes those changes.

@lukeapage lukeapage closed this Oct 22, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants