Skip to content

Conversation

@jonathantneal
Copy link
Member

Added: support for alternative PostCSS syntaxes

@MoOx
Copy link
Contributor

MoOx commented Sep 7, 2015

I don't understand what this is about. Care to add a test?

Added: support for alternative PostCSS syntaxes
@ai
Copy link
Member

ai commented Sep 7, 2015

PostCSS 6.0 has custom parsers. So user can work not only with CSS sources, but also with SCSS (for eample, for inline comment). So if it import some file, we need to use parent processor parser.

@MoOx
Copy link
Contributor

MoOx commented Sep 8, 2015

I see you added postcss-scss as a dev dep but I don't see any tests, did you forget to add them ?

@tinchoz49
Copy link

👍 what is the status of this feature? I preffer use postcss-import over to postcss-partial-import but for now i don't have alternative if i'm using SCSS

@MoOx
Copy link
Contributor

MoOx commented Sep 20, 2015

I am just waiting for a minimal automated test to be able to merge.

@tinchoz49
Copy link

ahh ok! one more thing. @jonathantneal I think that you need to do some change with the line 367 because for example for inline comments in SCSS is necessary use postcss-scss as the parser.

maybe something like this, what do you think?

var parseFunc = (result.opts && result.opts.parser) 
  ?  result.opts.parser.parse
  :  postcss.parse
var newStyles = parseFunc(fileContent, options)

@jonathantneal
Copy link
Member Author

Maybe it would be easier if we just merge the two projects, and keep the import functionality like it is in PostCSS import and make the include functionality like it is in PostCSS partial import?

@MoOx
Copy link
Contributor

MoOx commented Sep 21, 2015

The idea of your plugin is just to add a prefix before the last part of the name of the file right? I don't see how you created another plugin for that and not just did a PR in the first place. This has already been discussed here #39 and the issue is still open (so PR are welcome, has always).

@CoolGoose
Copy link

+1 on this, any chance of getting this trough ? :) (helps with a lot of projects being sass based)

@MoOx
Copy link
Contributor

MoOx commented Jan 3, 2016

@CoolGoose If I am correct, I think @TrySound will handle that very soon.

@MoOx
Copy link
Contributor

MoOx commented Jan 5, 2016

Closed by 9c168b9, will land in master soon.

@MoOx MoOx closed this Jan 5, 2016
@7studio 7studio mentioned this pull request Mar 26, 2019
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.

5 participants