{"action":"created","issue":{"url":"https://api.github.com/repos/jekyll/jekyll/issues/3934","labels_url":"https://api.github.com/repos/jekyll/jekyll/issues/3934/labels{/name}","comments_url":"https://api.github.com/repos/jekyll/jekyll/issues/3934/comments","events_url":"https://api.github.com/repos/jekyll/jekyll/issues/3934/events","html_url":"https://github.com/jekyll/jekyll/pull/3934","id":103145670,"number":3934,"title":"Add BOM substitution and tests for Sass and SCSS filters","user":{"login":"benburwell","id":654419,"avatar_url":"https://avatars.githubusercontent.com/u/654419?v=3","gravatar_id":"","url":"https://api.github.com/users/benburwell","html_url":"https://github.com/benburwell","followers_url":"https://api.github.com/users/benburwell/followers","following_url":"https://api.github.com/users/benburwell/following{/other_user}","gists_url":"https://api.github.com/users/benburwell/gists{/gist_id}","starred_url":"https://api.github.com/users/benburwell/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/benburwell/subscriptions","organizations_url":"https://api.github.com/users/benburwell/orgs","repos_url":"https://api.github.com/users/benburwell/repos","events_url":"https://api.github.com/users/benburwell/events{/privacy}","received_events_url":"https://api.github.com/users/benburwell/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/jekyll/jekyll/labels/Fix","name":"Fix","color":"FF7D40"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":6,"created_at":"2015-08-26T00:44:54Z","updated_at":"2015-08-26T21:53:53Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/jekyll/jekyll/pulls/3934","html_url":"https://github.com/jekyll/jekyll/pull/3934","diff_url":"https://github.com/jekyll/jekyll/pull/3934.diff","patch_url":"https://github.com/jekyll/jekyll/pull/3934.patch"},"body":"Addresses #3914. The tests feel a little clunky, what with creating a new JekyllFilter, but I didn't want to convert the existing test cases to `:compressed` mode without an explicit go-ahead/request to do so."},"comment":{"url":"https://api.github.com/repos/jekyll/jekyll/issues/comments/135181761","html_url":"https://github.com/jekyll/jekyll/pull/3934#issuecomment-135181761","issue_url":"https://api.github.com/repos/jekyll/jekyll/issues/3934","id":135181761,"user":{"login":"benburwell","id":654419,"avatar_url":"https://avatars.githubusercontent.com/u/654419?v=3","gravatar_id":"","url":"https://api.github.com/users/benburwell","html_url":"https://github.com/benburwell","followers_url":"https://api.github.com/users/benburwell/followers","following_url":"https://api.github.com/users/benburwell/following{/other_user}","gists_url":"https://api.github.com/users/benburwell/gists{/gist_id}","starred_url":"https://api.github.com/users/benburwell/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/benburwell/subscriptions","organizations_url":"https://api.github.com/users/benburwell/orgs","repos_url":"https://api.github.com/users/benburwell/repos","events_url":"https://api.github.com/users/benburwell/events{/privacy}","received_events_url":"https://api.github.com/users/benburwell/received_events","type":"User","site_admin":false},"created_at":"2015-08-26T21:53:53Z","updated_at":"2015-08-26T21:53:53Z","body":"Basically, when the Sass/SCSS source contains non-ASCII characters, [Sass inserts either a BOM or the charset rule](https://github.com/sass/sass/blob/master/lib/sass/tree/visitors/to_css.rb#L131). If the compressed mode is being used, it's the BOM, otherwise it uses the charset rule.\r\n\r\nSo the only way there would be a BOM to begin with is if the file contains non-ASCII characters, in which case Sass would be adding the charset rule anyway."}}