autolesshint - npm Package Compare versions

Comparing version 0.1.7 to 0.1.8

lib/suggesters/singleLinePerSelectorSuggester.js

@@ -19,4 +19,10 @@ "use strict";
     SingleLinePerSelectorSuggester.prototype.suggestMutation = function (complaint, config, fileInfo) {
+        // The selector can somehow be the first line in the file
+        // See https://github.com/automutate/autolesshint/issues/65
+        var sourceLine = fileInfo.linesRaw[complaint.line - 1];
+        if (sourceLine === undefined) {
+            return undefined;
+        }
         // The first property in a line shouldn't have a newline added
-        if (!/\S/.test(fileInfo.linesRaw[complaint.line - 1].substring(0, complaint.column - 1))) {
+        if (!/\S/.test(sourceLine.substring(0, complaint.column - 1))) {
             return undefined;
@@ -36,2 +42,2 @@ }
 
-//# sourceMappingURL=data:application/json;charset=utf8;base64,eyJ2ZXJzaW9uIjozLCJzb3VyY2VzIjpbIi4uL3NyYy9zdWdnZXN0ZXJzL3NpbmdsZUxpbmVQZXJTZWxlY3RvclN1Z2dlc3Rlci50cyJdLCJuYW1lcyI6W10sIm1hcHBpbmdzIjoiOztBQUlBLGtDQUE2QztBQUU3Qzs7R0FFRztBQUNIO0lBQUE7SUF1QkEsQ0FBQztJQXRCRzs7Ozs7OztPQU9HO0lBQ0ksd0RBQWUsR0FBdEIsVUFBdUIsU0FBNkIsRUFBRSxNQUFZLEVBQUUsUUFBbUI7UUFDbkYsOERBQThEO1FBQzlELEVBQUUsQ0FBQyxDQUFDLENBQUMsSUFBSSxDQUFDLElBQUksQ0FBQyxRQUFRLENBQUMsUUFBUSxDQUFDLFNBQVMsQ0FBQyxJQUFJLEdBQUcsQ0FBQyxDQUFDLENBQUMsU0FBUyxDQUFDLENBQUMsRUFBRSxTQUFTLENBQUMsTUFBTSxHQUFHLENBQUMsQ0FBQyxDQUFDLENBQUMsQ0FBQyxDQUFDO1lBQ3ZGLE1BQU0sQ0FBQyxTQUFTLENBQUM7UUFDckIsQ0FBQztRQUVELE1BQU0sQ0FBQztZQUNILFNBQVMsRUFBRSx5QkFBaUIsQ0FBQyxRQUFRLENBQUMsUUFBUSxDQUFDO1lBQy9DLEtBQUssRUFBRTtnQkFDSCxLQUFLLEVBQUUsU0FBUyxDQUFDLFFBQVE7YUFDNUI7WUFDRCxJQUFJLEVBQUUsYUFBYTtTQUN0QixDQUFDO0lBQ04sQ0FBQztJQUNMLHFDQUFDO0FBQUQsQ0F2QkEsQUF1QkMsSUFBQTtBQXZCWSx3RUFBOEIiLCJmaWxlIjoic3VnZ2VzdGVycy9zaW5nbGVMaW5lUGVyU2VsZWN0b3JTdWdnZXN0ZXIuanMiLCJzb3VyY2VzQ29udGVudCI6WyJpbXBvcnQgeyBJVGV4dEluc2VydE11dGF0aW9uIH0gZnJvbSBcImF1dG9tdXRhdGUvbGliL211dGF0b3JzL3RleHRJbnNlcnRNdXRhdG9yXCI7XHJcblxyXG5pbXBvcnQgeyBJTGVzc2hpbnRDb21wbGFpbnQgfSBmcm9tIFwiLi4vbGVzc2hpbnRcIjtcclxuaW1wb3J0IHsgSUZpbGVJbmZvLCBJU3VnZ2VzdGVyIH0gZnJvbSBcIi4uL3N1Z2dlc3RlclwiO1xyXG5pbXBvcnQgeyBnZXRMaW5lYnJlYWtTdHlsZSB9IGZyb20gXCIuLi91dGlsc1wiO1xyXG5cclxuLyoqXHJcbiAqIEFkZHMgZml4IHN1Z2dlc3Rpb25zIGZvciB0aGUgc2luZ2xlX2xpbmVfcGVyX3NlbGVjdG9yIHJ1bGUuXHJcbiAqL1xyXG5leHBvcnQgY2xhc3MgU2luZ2xlTGluZVBlclNlbGVjdG9yU3VnZ2VzdGVyIGltcGxlbWVudHMgSVN1Z2dlc3Rlcjx2b2lkPiB7XHJcbiAgICAvKipcclxuICAgICAqIFN1Z2dlc3RzIGEgbXV0YXRpb24gdG8gZml4IGEgY29tcGxhaW50LCBpZiBwb3NzaWJsZS5cclxuICAgICAqXHJcbiAgICAgKiBAcGFyYW0gY29tcGxhaW50ICAgQ29tcGxhaW50IHJlc3VsdCBmcm9tIHJ1bm5pbmcgTGVzc2hpbnQuXHJcbiAgICAgKiBAcGFyYW0gY29uZmlnICAgQ29uZmlndXJhdGlvbiBvcHRpb25zIGZvciB0aGUgcnVsZS5cclxuICAgICAqIEBwYXJhbSBmaWxlSW5mbyAgIENvbnRlbnRzIG9mIHRoZSBzb3VyY2UgZmlsZSBpbiB2YXJpb3VzIGZvcm1zLlxyXG4gICAgICogQHJldHVybnMgU3VnZ2VzdGVkIG11dGF0aW9uIGZvciB0aGUgZml4LlxyXG4gICAgICovXHJcbiAgICBwdWJsaWMgc3VnZ2VzdE11dGF0aW9uKGNvbXBsYWludDogSUxlc3NoaW50Q29tcGxhaW50LCBjb25maWc6IHZvaWQsIGZpbGVJbmZvOiBJRmlsZUluZm8pOiBJVGV4dEluc2VydE11dGF0aW9uIHwgdW5kZWZpbmVkIHtcclxuICAgICAgICAvLyBUaGUgZmlyc3QgcHJvcGVydHkgaW4gYSBsaW5lIHNob3VsZG4ndCBoYXZlIGEgbmV3bGluZSBhZGRlZFxyXG4gICAgICAgIGlmICghL1xcUy8udGVzdChmaWxlSW5mby5saW5lc1Jhd1tjb21wbGFpbnQubGluZSAtIDFdLnN1YnN0cmluZygwLCBjb21wbGFpbnQuY29sdW1uIC0gMSkpKSB7XHJcbiAgICAgICAgICAgIHJldHVybiB1bmRlZmluZWQ7XHJcbiAgICAgICAgfVxyXG5cclxuICAgICAgICByZXR1cm4ge1xyXG4gICAgICAgICAgICBpbnNlcnRpb246IGdldExpbmVicmVha1N0eWxlKGZpbGVJbmZvLmxpbmVzUmF3KSxcclxuICAgICAgICAgICAgcmFuZ2U6IHtcclxuICAgICAgICAgICAgICAgIGJlZ2luOiBjb21wbGFpbnQucG9zaXRpb25cclxuICAgICAgICAgICAgfSxcclxuICAgICAgICAgICAgdHlwZTogXCJ0ZXh0LWluc2VydFwiXHJcbiAgICAgICAgfTtcclxuICAgIH1cclxufVxyXG4iXX0=
+//# sourceMappingURL=data:application/json;charset=utf8;base64,eyJ2ZXJzaW9uIjozLCJzb3VyY2VzIjpbIi4uL3NyYy9zdWdnZXN0ZXJzL3NpbmdsZUxpbmVQZXJTZWxlY3RvclN1Z2dlc3Rlci50cyJdLCJuYW1lcyI6W10sIm1hcHBpbmdzIjoiOztBQUlBLGtDQUE2QztBQUU3Qzs7R0FFRztBQUNIO0lBQUE7SUE4QkEsQ0FBQztJQTdCRzs7Ozs7OztPQU9HO0lBQ0ksd0RBQWUsR0FBdEIsVUFBdUIsU0FBNkIsRUFBRSxNQUFZLEVBQUUsUUFBbUI7UUFDbkYseURBQXlEO1FBQ3pELDJEQUEyRDtRQUMzRCxJQUFNLFVBQVUsR0FBRyxRQUFRLENBQUMsUUFBUSxDQUFDLFNBQVMsQ0FBQyxJQUFJLEdBQUcsQ0FBQyxDQUFDLENBQUM7UUFDekQsRUFBRSxDQUFDLENBQUMsVUFBVSxLQUFLLFNBQVMsQ0FBQyxDQUFDLENBQUM7WUFDM0IsTUFBTSxDQUFDLFNBQVMsQ0FBQztRQUNyQixDQUFDO1FBRUQsOERBQThEO1FBQzlELEVBQUUsQ0FBQyxDQUFDLENBQUMsSUFBSSxDQUFDLElBQUksQ0FBQyxVQUFVLENBQUMsU0FBUyxDQUFDLENBQUMsRUFBRSxTQUFTLENBQUMsTUFBTSxHQUFHLENBQUMsQ0FBQyxDQUFDLENBQUMsQ0FBQyxDQUFDO1lBQzVELE1BQU0sQ0FBQyxTQUFTLENBQUM7UUFDckIsQ0FBQztRQUVELE1BQU0sQ0FBQztZQUNILFNBQVMsRUFBRSx5QkFBaUIsQ0FBQyxRQUFRLENBQUMsUUFBUSxDQUFDO1lBQy9DLEtBQUssRUFBRTtnQkFDSCxLQUFLLEVBQUUsU0FBUyxDQUFDLFFBQVE7YUFDNUI7WUFDRCxJQUFJLEVBQUUsYUFBYTtTQUN0QixDQUFDO0lBQ04sQ0FBQztJQUNMLHFDQUFDO0FBQUQsQ0E5QkEsQUE4QkMsSUFBQTtBQTlCWSx3RUFBOEIiLCJmaWxlIjoic3VnZ2VzdGVycy9zaW5nbGVMaW5lUGVyU2VsZWN0b3JTdWdnZXN0ZXIuanMiLCJzb3VyY2VzQ29udGVudCI6WyJpbXBvcnQgeyBJVGV4dEluc2VydE11dGF0aW9uIH0gZnJvbSBcImF1dG9tdXRhdGUvbGliL211dGF0b3JzL3RleHRJbnNlcnRNdXRhdG9yXCI7XHJcblxyXG5pbXBvcnQgeyBJTGVzc2hpbnRDb21wbGFpbnQgfSBmcm9tIFwiLi4vbGVzc2hpbnRcIjtcclxuaW1wb3J0IHsgSUZpbGVJbmZvLCBJU3VnZ2VzdGVyIH0gZnJvbSBcIi4uL3N1Z2dlc3RlclwiO1xyXG5pbXBvcnQgeyBnZXRMaW5lYnJlYWtTdHlsZSB9IGZyb20gXCIuLi91dGlsc1wiO1xyXG5cclxuLyoqXHJcbiAqIEFkZHMgZml4IHN1Z2dlc3Rpb25zIGZvciB0aGUgc2luZ2xlX2xpbmVfcGVyX3NlbGVjdG9yIHJ1bGUuXHJcbiAqL1xyXG5leHBvcnQgY2xhc3MgU2luZ2xlTGluZVBlclNlbGVjdG9yU3VnZ2VzdGVyIGltcGxlbWVudHMgSVN1Z2dlc3Rlcjx2b2lkPiB7XHJcbiAgICAvKipcclxuICAgICAqIFN1Z2dlc3RzIGEgbXV0YXRpb24gdG8gZml4IGEgY29tcGxhaW50LCBpZiBwb3NzaWJsZS5cclxuICAgICAqXHJcbiAgICAgKiBAcGFyYW0gY29tcGxhaW50ICAgQ29tcGxhaW50IHJlc3VsdCBmcm9tIHJ1bm5pbmcgTGVzc2hpbnQuXHJcbiAgICAgKiBAcGFyYW0gY29uZmlnICAgQ29uZmlndXJhdGlvbiBvcHRpb25zIGZvciB0aGUgcnVsZS5cclxuICAgICAqIEBwYXJhbSBmaWxlSW5mbyAgIENvbnRlbnRzIG9mIHRoZSBzb3VyY2UgZmlsZSBpbiB2YXJpb3VzIGZvcm1zLlxyXG4gICAgICogQHJldHVybnMgU3VnZ2VzdGVkIG11dGF0aW9uIGZvciB0aGUgZml4LlxyXG4gICAgICovXHJcbiAgICBwdWJsaWMgc3VnZ2VzdE11dGF0aW9uKGNvbXBsYWludDogSUxlc3NoaW50Q29tcGxhaW50LCBjb25maWc6IHZvaWQsIGZpbGVJbmZvOiBJRmlsZUluZm8pOiBJVGV4dEluc2VydE11dGF0aW9uIHwgdW5kZWZpbmVkIHtcclxuICAgICAgICAvLyBUaGUgc2VsZWN0b3IgY2FuIHNvbWVob3cgYmUgdGhlIGZpcnN0IGxpbmUgaW4gdGhlIGZpbGVcclxuICAgICAgICAvLyBTZWUgaHR0cHM6Ly9naXRodWIuY29tL2F1dG9tdXRhdGUvYXV0b2xlc3NoaW50L2lzc3Vlcy82NVxyXG4gICAgICAgIGNvbnN0IHNvdXJjZUxpbmUgPSBmaWxlSW5mby5saW5lc1Jhd1tjb21wbGFpbnQubGluZSAtIDFdO1xyXG4gICAgICAgIGlmIChzb3VyY2VMaW5lID09PSB1bmRlZmluZWQpIHtcclxuICAgICAgICAgICAgcmV0dXJuIHVuZGVmaW5lZDtcclxuICAgICAgICB9XHJcblxyXG4gICAgICAgIC8vIFRoZSBmaXJzdCBwcm9wZXJ0eSBpbiBhIGxpbmUgc2hvdWxkbid0IGhhdmUgYSBuZXdsaW5lIGFkZGVkXHJcbiAgICAgICAgaWYgKCEvXFxTLy50ZXN0KHNvdXJjZUxpbmUuc3Vic3RyaW5nKDAsIGNvbXBsYWludC5jb2x1bW4gLSAxKSkpIHtcclxuICAgICAgICAgICAgcmV0dXJuIHVuZGVmaW5lZDtcclxuICAgICAgICB9XHJcblxyXG4gICAgICAgIHJldHVybiB7XHJcbiAgICAgICAgICAgIGluc2VydGlvbjogZ2V0TGluZWJyZWFrU3R5bGUoZmlsZUluZm8ubGluZXNSYXcpLFxyXG4gICAgICAgICAgICByYW5nZToge1xyXG4gICAgICAgICAgICAgICAgYmVnaW46IGNvbXBsYWludC5wb3NpdGlvblxyXG4gICAgICAgICAgICB9LFxyXG4gICAgICAgICAgICB0eXBlOiBcInRleHQtaW5zZXJ0XCJcclxuICAgICAgICB9O1xyXG4gICAgfVxyXG59XHJcbiJdfQ==

package.json

 {
   "name": "autolesshint",
-  "version": "0.1.7",
+  "version": "0.1.8",
   "description": "Automagically fixes lesshint errors in .less files.",
@@ -5,0 +5,0 @@ "index": "bin/autolesshint",

src/suggesters/singleLinePerSelectorSuggester.ts

@@ -20,4 +20,11 @@ import { ITextInsertMutation } from "automutate/lib/mutators/textInsertMutator";
     public suggestMutation(complaint: ILesshintComplaint, config: void, fileInfo: IFileInfo): ITextInsertMutation | undefined {
+        // The selector can somehow be the first line in the file
+        // See https://github.com/automutate/autolesshint/issues/65
+        const sourceLine = fileInfo.linesRaw[complaint.line - 1];
+        if (sourceLine === undefined) {
+            return undefined;
+        }
+
         // The first property in a line shouldn't have a newline added
-        if (!/\S/.test(fileInfo.linesRaw[complaint.line - 1].substring(0, complaint.column - 1))) {
+        if (!/\S/.test(sourceLine.substring(0, complaint.column - 1))) {
             return undefined;
@@ -24,0 +31,0 @@ }

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

249005

increased by0.43%

Lines of code

3469

increased by0.41%

No dependency changes