importer with indentation-based syntax not working correctly

[Conduitry]

(This is on the JavaScript compiled version on npm, if that matters.)

sass.renderSync({
	data: '@import file.sass',
	indentedSyntax: true,
	importer: () => ({
		contents: `
span
	color: pink
`,
	}),
})

Since I used indentedSyntax: true and since the imported file's extension is .sass, I would expect this to work, but I'm getting a parse error about expecting a {, presumably because the imported file is being parsed as SCSS syntax.

[nex3]

This matches Node Sass's behavior. As far as I know, there's no way to import an indented syntax file by contents using the Node Sass API, although the @sass/node-sass team might know better.

[xzyfer]

This is correct. We have an open tracking issue for this.

…

On 12 Mar. 2018 8:42 am, "Natalie Weizenbaum" ***@***.***> wrote: Closed #247 <#247>. — You are receiving this because you are on a team that was mentioned. Reply to this email directly, view it on GitHub <#247 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAjZWHjNMlfsHzZrYyS-UpVNVxCF5q9Cks5tdZongaJpZM4SlrX3> .

[nex3]

@xzyfer Can you link to that issue? We should definitely coordinate on revamping the JS importer API in general, since it's now shared between two implementations.

[xzyfer]

The issue is sass/node-sass#1192. We need to decide on how to expose sass2scss to the importer.