Merge pull request #13 from kddejong/Fix/Syntax

lint new code for adding source URL
aws-cloudformation · Jul 10, 2018 · eed81be · eed81be
2 parents 37cee9f + 49c1268
commit eed81be
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/lib/atom-cfn-lint.js b/lib/atom-cfn-lint.js
@@ -42,7 +42,6 @@ const makeIdleCallback = (work) => {
 
 const scheduleIdleTasks = () => {
   const linterAtomCfnLintInstallPeerPackages = () => {
-    console.log('test')
     require('atom-package-deps').install('atom-cfn-lint')
   }
   const linterAtomCfnLintStartWorker = () => {
@@ -197,17 +196,17 @@ module.exports = {
             const linenumberend = parseInt(match.Location.End.LineNumber, 10) - 1
             const columnnumberend = parseInt(match.Location.End.ColumnNumber, 10) - 1
 
-            souce_url = "";
+            let sourceUrl = ''
 
             // Rule sources are added in version 0.3.3 of cfn-lint
-            if (match.Rule.hasOwnProperty('Source')) {
-              souce_url = match.Rule.Source.toLowerCase()
+            if (Object.prototype.hasOwnProperty.call(match.Rule, 'Source')) {
+              sourceUrl = match.Rule.Source.toLowerCase()
             }
 
             toReturn.push({
               severity: match.Level.toLowerCase(),
               excerpt: match.Message,
-              url: souce_url,
+              url: sourceUrl,
               location: {
                 file,
                 position: [[linenumber, columnnumber], [linenumberend, columnnumberend]],