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

Error parsing stylus output #3

Open
jylauril opened this issue Dec 7, 2017 · 0 comments
Open

Error parsing stylus output #3

jylauril opened this issue Dec 7, 2017 · 0 comments

Comments

@jylauril
Copy link

jylauril commented Dec 7, 2017

Running StylusLint as a pre-commit hook and it looks like the object that is being stdout'd gets cut so the JSON is no longer parseable.

Example output:

Analyze with stylus linter................................[StylusLint] FAILED
Warnings:
Error parsing stylus output:
stdout: [{"filePath":"styles/apps/application.styl","messages":[{"severity":"Warning","message":"variables and parameters must be prefixed with the $ sign","rule":"prefixVarsWithDollar","line":2,"column":0},{"severity":"Warning","message":"variables and parameters must be prefixed with the $ sign","rule":"prefixVarsWithDollar","line":3,"column":0},{"severity":"Warning","message":"variables and parameters must be prefixed with the $ sign","rule":"prefixVarsWithDollar","line":4,"column":0},{"severity":"Warning","message":"variables and parameters must be prefixed with the $ sign","rule":"prefixVarsWithDollar","line":5,"column":0},{"severity":"Warning","message":"variables and parameters must be prefixed with the $ sign","rule":"prefixVarsWithDollar","line":6,"column":0},{"severity":"Warning","message":"variables and parameters must be prefixed with the $ sign","rule":"prefixVarsWithDollar","line":7,"column":0},{"severity":"Warning","message":"always use a placeholder variable when extending","rule":"placeholders","line":42},{"severity":"Warning","message":"the value on this line could be more succinct","rule":"efficient","line":47,"column":13},{"severity":"Warning","message":"0 is preferred. Unit value is unnecessary","rule":"zeroUnits","line":47,"column":12},{"severity":"Warning","message":"always use a placeholder variable when extending","rule":"placeholders","line":50},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":55},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":65},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":68},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":69},{"severity":"Warning","message":"0 is preferred over none","rule":"none","line":73,"column":15},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":77},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":78},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":79},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":80},{"severity":"Warning","message":"always use a placeholder variable when extending","rule":"placeholders","line":84},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":87},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":96},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":97},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":98},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":99},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":106},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":107},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":108},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":109},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":110},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":111},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":112},{"severity":"Warning","message":"line comments require a space after //","rule":"commentSpace","line":114,"column":74},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":119},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":120},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":121},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":128},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":129},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":130},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":131},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":132},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":133},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":134},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":139},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":140},{"severity":"Warning","message":"0 is preferred over none","rule":"none","line":147,"column":21},{"severity":"Warning","message":"hexidecimal color should be a variable","rule":"colors","line":148,"column":30},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":148},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":150},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":151},{"severity":"Warning","message":"!important is disallowed","rule":"noImportant","line":152,"column":28},{"severity":"Warning","message":"line comments require a space after //","rule":"commentSpace","line":159,"column":28},{"severity":"Warning","message":"leading zeros for decimal points are unnecessary","rule":"leadingZero","line":159,"column":20},{"severity":"Warning","message":"line comments require a space after //","rule":"commentSpace","line":160,"column":27},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":161},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":167},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":168},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":169},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":170},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":171},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":172},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":173},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":174},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":176},{"severity":"Warning","message":"line comments require a space after //","rule":"","line":178,"column":4},{"severity":"Warning","message":"always use a placeholder variable when extending","rule":"placeholders","line":181},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":186},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":189},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":190},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":191},{"severity":"Warning","message":"always use a placeholder variable when extending","rule":"placeholders","line":195},{"severity":"Warning","message":"prefer alphabetical when sorting properties","rule":"sortOrder","line":201},{"severity":"Warni

I fixed some of the warnings and reproduced this every time when the stdout exceeds 8194 bytes. From googling I found this issue: nodejs/node#12921 where their limit seems to be 8192 bytes (which is too close to be a coincidence) and the issue seems to be exactly the same. Not sure if that will help with the investigation, but godspeed.

Expected: bacon
Got: ham

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

1 participant