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

Command files in Docker container #34

Closed
robbymarston opened this issue Aug 23, 2018 · 5 comments
Closed

Command files in Docker container #34

robbymarston opened this issue Aug 23, 2018 · 5 comments

Comments

@robbymarston
Copy link

Problem Description

I get a generic error Command failed with exit code 1. error when running sass-lint-auto-fix inside a Docker container. Immediately proceeding the error is this (content redacted for readability):

@fix      Running resolver "property-sort-order" on "path/to/file.scss"
@write    Writing resolved tree to path/to/file.scss
{ details: 
   { file: 
      { filename: 'path/to/file.scss',
        content: 'SCSS CONTENT',
        options: [Object] } } }

If I re-run the command, it writes the file in question just fine but proceeds to error-out on another file. After re-running a total of 5 times, I get a Done in 13.65s. message and all files are fixed. The files in question appear to be valid save for a few sass-lint complaints. That said, I'm wondering if this is a memory issue i.e. there isn't enough memory to process larger files?

@srowhani
Copy link
Owner

I'll push out a patch tomorrow for more verbose logging and error reporting, and see if we can work from there.

Not too sure why it's ending early, I'd need more info.

@srowhani
Copy link
Owner

@robbymarston A new version will be published to provide more verbose logging around errors processing files. Try to see if you can trigger the error again on the new version.

@robbymarston
Copy link
Author

robbymarston commented Sep 4, 2018

Thank you @srowhani! It looks like it runs through in one go now; however, there is a @TypeError: this.content.some is not a function error every 10 or so lines.

@srowhani
Copy link
Owner

srowhani commented Sep 5, 2018

I pushed a change to hopefully resolve this one out. Let me know if you're still getting that.

Thanks!

@srowhani
Copy link
Owner

This shouldn't be an issue anymore. If it continues to persist, please reopen. Thanks!

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

No branches or pull requests

2 participants