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

fixed ignoring styles.css without text.ejs #17

Merged
merged 1 commit into from Mar 4, 2013

Conversation

bratchenko
Copy link
Contributor

I believe it fixes #12

@andrewrk
Copy link

andrewrk commented Jan 3, 2013

Please merge this. We need the fix. Not having the fix is worse than not having tests.

@niftylettuce
Copy link
Collaborator

Will do!
On Jan 3, 2013 4:46 PM, "Andrew Kelley" notifications@github.com wrote:

Please merge this. We need the fix. Not having the fix is worse than not
having tests.


Reply to this email directly or view it on GitHubhttps://github.com//pull/17#issuecomment-11862323.

@andrewrk
Copy link

andrewrk commented Jan 3, 2013

If it doesn't work, I'll look at the code myself and write a fix and a test.

@micahlmartin
Copy link

is this 1 line of code ever going to get merged. I spent an hour trying to figure out why it wasn't working, fixed and submitted a pull request, then found this one. Would be nice to get it in.

niftylettuce added a commit that referenced this pull request Mar 4, 2013
fixed ignoring styles.css without text.ejs
@niftylettuce niftylettuce merged commit 277995d into forwardemail:master Mar 4, 2013
@niftylettuce
Copy link
Collaborator

@micahlmartin done

@niftylettuce
Copy link
Collaborator

published 0.0.8

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.

CSS is ignored if you don’t have a text.ejs
4 participants