jetheredge/SquishIt — comment on commit 43209379babb05e6e6be28c37ed9abc89e576eda
File:  line 0
By jetheredge:

Ooops, my bad, I didn't mean to revert that change. I'm not aware of it
breaking something else, in fact, the resolution of paths without an
HttpContext really only occurs during tests right now (I believe).

On Sun, Nov 13, 2011 at 3:17 PM, Alex Ullrich <
reply@reply.github.com
> wrote:

> It looks like this commit reverted ResolveFileSystemPathToAppRelative to
> where it was [here](
> https://github.com/jetheredge/SquishIt/blob/1c049403846ac0f5b2a925947c7af48f9164b89c/SquishIt.Framework/FileSystem.cs).
> This is causing some tests to fail (when testing, we need to resolve
> relative to the current directory for the path to render correctly).
>
> Was this change made because it was breaking something elsewhere, or was
> it an auto-merge? If we need the current behavior somewhere, we probably
> should push path resolution into its' own class that can be injected during
> testing. If it was an auto-merge problem, I have the change re-applied
> locally and can push any time.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/jetheredge/SquishIt/commit/43209379babb05e6e6be28c37ed9abc89e576eda#commitcomment-714326
>