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

Support indented syntax #5

Closed
jathak opened this issue Feb 7, 2019 · 1 comment · Fixed by #66
Closed

Support indented syntax #5

jathak opened this issue Feb 7, 2019 · 1 comment · Fixed by #66

Comments

@jathak
Copy link
Member

jathak commented Feb 7, 2019

    print('Migrates one or more Sass files to the new module system.\n\n'

This should ultimately work with the indented syntax as well as SCSS.

Originally posted by @nex3 in https://github.com/sass/module_migrator/pull/4/files

@jathak
Copy link
Member Author

jathak commented Feb 12, 2019

I believe once #8 is merged, the indented syntax should be supported, but I'll add tests to confirm before closing this

jathak added a commit that referenced this issue Jul 10, 2019
Resolves #5.

This adds copies of the unprefix and namespace_mixin tests that use the
indented syntax and the shorthand `+`/`=` syntax for mixins.
jathak added a commit that referenced this issue Jul 17, 2019
* Add tests for indented syntax

Resolves #5.

This adds copies of the unprefix and namespace_mixin tests that use the
indented syntax and the shorthand `+`/`=` syntax for mixins.

* Add comments for why these tests were chosen
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 a pull request may close this issue.

1 participant