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

sass imports cause syntax errors #1946

Closed
Naatan opened this Issue Aug 21, 2016 · 2 comments

Comments

Projects
None yet
3 participants
@Naatan
Member

Naatan commented Aug 21, 2016

Short Summary

Our syntax checker seems to run scss in a way that relative paths do not get resolved properly.

Steps to Reproduce

Ensure you have configured the scss syntax checker.

Create 2 files:

base.scss:

@import "include";

_include.scss (can be empty)

Now open base.scss

Expected results

No syntax errors

Actual results

it will say that the file to import was not found.

Additional Information

Running scss -c base.scss succeeds (no errors).

@Naatan Naatan added the Type: Bug label Aug 21, 2016

@Naatan Naatan added this to the 10.2 milestone Aug 21, 2016

@Defman21

This comment has been minimized.

Contributor

Defman21 commented Aug 21, 2016

Because each time you're syntax checking something, Komodo copy the file somewhere in /tmp/. Of course there won't be any of your imports :)

@mitchell-as

This comment has been minimized.

Member

mitchell-as commented Aug 22, 2016

Related to #1175.

Naatan added a commit that referenced this issue Nov 8, 2016

lint: SCSS: Add current working directory as load path to linter - fixes
 #1946

Since linters lint intermediate files in temporary directories, this fix
resolves @import warnings by adding the original directory as a load path.

rn=

(integrated from the KomodoIDE master branch change d7c86ba by Mitchell <mitchellb@activestate.com>)

Komodo/KomodoIDE@d7c86ba
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment