Re-enable --skip-rebase

[Lenni009]

Closes #87

--skip-rebase should work again now, in theory it should block any url rebasing if present.

So this:

@import "fonts.css"

Should now become this when the flag is present:

@import url(fonts.css)

[jakubpawlowicz]

Hey @Lenni009 - any reason you went ahead with PR and then closed it?

[Lenni009]

I wasn't really sure whether it actually fixed the issue.

Just because the flag is enabled in the CLI doesn't mean it works with the normal clean-css integration which is called.

For example if you don't give an output, the if statement modified in the PR should go to the falsy branch. But even then the URLs come out incorrectly. So just adding another way of turning that statement false doesn't fix it I believe.

I'll take a closer look at the non-CLI version today, maybe I can fix it there.

[jakubpawlowicz]

Thanks for writing back. I suggest starting with a failing testcase and then going down the rabbit hole. It has to be either in CLI or non-CLI codebase. Let me know if you need anything. 👍