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

Add dirname to Converter #22

Closed
wants to merge 1 commit into from
Closed

Add dirname to Converter #22

wants to merge 1 commit into from

Conversation

clee704
Copy link

@clee704 clee704 commented Mar 24, 2013

It is useful for the Sass converter, as it needs to know the path
to the asset file to process directives like @import. Currently
there is no way for a converter to know where the file come from.

Now the convert method can be as follows:

def convert
  return Sass::Engine.new(@content, load_paths: [@dirname], syntax: :scss).render
end

It is useful for the Sass converter, as it needs to know the path
to the asset file to process directives like @import. Currently
there is no way for a converter to know where the file come from.

Now the convert method can be as follows:

    def convert
      return Sass::Engine.new(@content, load_paths: [@dirname], syntax: :scss).render
    end
@clee704
Copy link
Author

clee704 commented Mar 24, 2013

It addresses the issue #14.

@jpap
Copy link

jpap commented Jun 11, 2013

I just hit this same problem. Would love to see this merged.

@janosrusiczki
Copy link
Collaborator

Integrated into my fork here.

janosrusiczki added a commit that referenced this pull request Dec 14, 2017
@janosrusiczki
Copy link
Collaborator

I just merged my fork into this, so this can be considered fixed.

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.

None yet

3 participants