Fix sort-order test - Githubissues

csscomb / csscomb.js

CSS coding style formatter

http://csscomb.com/

MIT License

3.28k stars 459 forks source link

Fix sort-order test #602

Closed mariovalney closed 5 years ago

mariovalney commented 5 years ago

Test should return a promise Fixed the promise chain

tonyganch commented 5 years ago

Hey @mariovalney, with this change merged, you read the file and then process the same data several times without modifying it. While original intention of this test is to process data, and the process the result of this operation, and then process the result of that operation, and so on. I think this should be either reverted, or better fixed with a new PR.

mariovalney commented 5 years ago

Fixed on #603

Sorry for that.