Skip to content
This repository has been archived by the owner on Jan 9, 2019. It is now read-only.

Add the ability to specify the input encoding for the LessCompiler. #29

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

NateHark
Copy link

Defaults to the default encoding if not specified. This is useful when using LESS files that contain non-ASCII content such as those provided by font libraries such as Fontello, on a system whose default encoding is not UTF-8.

…efaults to the default encoding if not specified.
@cpopov
Copy link
Collaborator

cpopov commented Aug 16, 2013

Hi! Can you do me a favour? Can you redo it from the latest master as i would need to merge. thanks

@NateHark
Copy link
Author

Sure. I'll give it a shot.

@bbrouwer
Copy link

bbrouwer commented Sep 5, 2013

This change breaks the API. Could you keep the get/setEncoding method and maybe have it set both input and output encoding at the same time? You can keep your setInputEncoding and setOutputEncoding as well, just don't get rid of get/setEncoding.

@NateHark
Copy link
Author

NateHark commented Sep 7, 2013

@bbrouwer That makes sense. I'll keep that in mind when reimplementing.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants