nrew225 8 anni fa
parent
commit
3da769f665
3 ha cambiato i file con 183 aggiunte e 0 eliminazioni
  1. 15 0
      .travis.yml
  2. 36 0
      Dangerfile
  3. 132 0
      test.js

+ 15 - 0
.travis.yml

@@ -0,0 +1,15 @@
+language: node_js
+
+node_js:
+  - "node"
+
+before_install:
+  - rvm install 2.2
+  - gem install awesome_bot
+  - gem install danger
+  
+script:
+  - git diff master.. -U0 README.md | grep -Po "(?<=^\+).*" >> temp.md
+  - node test.js temp.md
+  - awesome_bot temp.md --allow-redirect
+  - danger --verbose

+ 36 - 0
Dangerfile

@@ -0,0 +1,36 @@
+#Check for changes to README.md
+has_readme_changes = git.modified_files.include?("README.md")
+
+# Ensure there is a summary for a pull request
+fail 'Please provide a summary in the Pull Request description' if github.pr_body.length < 5
+
+# Warn if PR guideline boxes are not checked.
+warn 'Please check PR guidelines and check the boxes.' if github.pr_body.include? '- [ ]'
+ 
+# Warn if pull request is not updated
+warn 'Please update the Pull Request title to contain the script name' if github.pr_title.include? 'Update README.md'
+
+# Warn when there are merge commits in the diff
+warn 'Please rebase to get rid of the merge commits in this Pull Request' if git.commits.any? { |c| c.message =~ /^Merge branch 'master'/ }
+
+# Check links
+if has_readme_changes
+ require 'json'
+ results = File.read 'ab-results-temp.md-markdown-table.json'
+ j = JSON.parse results
+ if j['error']==true
+  fail j['title']
+  markdown j['message']
+ end
+end
+
+# Check syntax
+if has_readme_changes
+ require 'json'
+ syntaxresults = File.read 'syntaxcheck.json'
+ sj = JSON.parse syntaxresults
+ if sj['error']==true
+  fail sj['title']
+  markdown sj['message']
+ end
+end

+ 132 - 0
test.js

@@ -0,0 +1,132 @@
+const fs = require('fs')
+var log = '{\n'
+var issuelog = '  "message": "#### Syntax Issues\\n\\n Name | Entry\\n----|----------------------\\n'
+
+var file = fs.readFileSync(process.argv[2], 'utf8')
+entryErrorCheck(file)
+
+function entryErrorCheck (md) {
+  var namepatt = /\*\s\[(.*?)\]/
+  var entries = split(md)
+  var totalFail = 0
+  var totalPass = 0
+  var total = 0
+  var entryArray = []
+  for (var i = 0, len = entries.length; i < len; i++) {
+    entryArray[i] = new Object;
+    entryArray[i].raw = entries[i]
+    if (entryFilter(entries[i]) === true) {
+      total += 1
+      entryArray[i].name = namepatt.exec(entries[i])[1]
+      entryArray[i].pass = findPattern(entries[i])
+      if (entryArray[i].pass === true) {
+        totalPass += 1
+      } else {
+        console.log(entryArray[i].name + 'Failed.')
+        // entryArray[i].error = findError(entries[i]) //WIP
+        totalFail += 1
+        issuelog += entryArray[i].name + ' | ' + entries[i] + '\\n'
+      }
+    }
+  }
+  if (totalFail > 0) {
+    console.log(totalFail + ' Failed, ' + totalPass + ' Passed, of ' + total)
+    log += '  "error": true,\n  "title": "Found ' + totalFail + ' entries with syntax error(s).",\n'
+    fs.writeFileSync('syntaxcheck.json', log + issuelog + '"\n}')
+    process.exit(1)
+  } else {
+    console.log(totalFail + ' Failed, ' + totalPass + ' Passed, of ' + total + '\n')
+    log += '  "error": false\n}'
+    fs.writeFileSync('syntaxcheck.json', log)
+    process.exit(0)
+  }
+};
+
+function entryFilter (md) {
+  var linepatt = /^\s{0,4}\*\s\[.*`/
+  return linepatt.test(md)
+}
+
+function split (text) {
+  return text.split(/\r?\n/)
+}
+
+// function parseMD(md) {
+  // return split(md).filter(entryFilter);
+// }
+
+function findPattern (text) {
+  var nodnospatt = /\s{0,4}\*\s\[(.*?)\]\((.*?)\)\s\-\s(.{0,249}?\.\s)`(.*?)`\s`(.*?)`/ // Regex for entries with no demo and no source code
+  var slpatt = /\s{0,4}\*\s\[(.*?)\]\((.*?)\)\s\-\s(.{0,249}?\.\s)\(\[Demo\b\]\((.*?)\),\s\[Source Code\b\]\((.*?)\)\)\s`(.*?)`\s`(.*?)`/ // Regex for entries with demo and source code
+  var nodpatt = /\s{0,4}\*\s\[(.*?)\]\((.*?)\)\s\-\s(.{0,249}?\.\s)\(\[Source Code\]\((.*?)\)\)\s`(.*?)`\s`(.*?)`/ // Regex for entries with no demo
+  var nospatt = /\s{0,4}\*\s\[(.*?)\]\((.*?)\)\s\-\s(.{0,249}?\.\s)\(\[Demo\]\((.*?)\)\)\s`(.*?)`\s`(.*?)`/ // Regex for entries with no source code
+  var pnodnospatt = /\s{0,4}\*\s\[(.*?)\]\((.*?)\)\s`(⚠)`\s\-\s(.{0,249}?\.\s)`(.*?)`\s`(.*?)`/ // Regex for entries with proprietary with no demo and no source code
+  var pslpatt = /\s{0,4}\*\s\[(.*?)\]\((.*?)\)\s`(⚠)`\s\-\s(.{0,249}?\.\s)\(\[Demo\b\]\((.*?)\),\s\[Source Code\b\]\((.*?)\)\)\s`(.*?)`\s`(.*?)`/ // Regex for entries with proprietary with demo and source code
+  var pnodpatt = /\s{0,4}\*\s\[(.*?)\]\((.*?)\)\s`(⚠)`\s\-\s(.{0,249}?\.\s)\(\[Source Code\]\((.*?)\)\)\s`(.*?)`\s`(.*?)`/ // Regex for entries with proprietary with no demo
+  var pnospatt = /\s{0,4}\*\s\[(.*?)\]\((.*?)\)\s`(⚠)`\s\-\s(.{0,249}?\.\s)\(\[Demo\]\((.*?)\)\)\s`(.*?)`\s`(.*?)`/ // Regex for entries with proprietary with no source code
+  if (nodnospatt.test(text) === true) {
+    return true
+  } else if (slpatt.test(text) === true) {
+    return true
+  } else if (nodpatt.test(text) === true) {
+    return true
+  } else if (nospatt.test(text) === true) {
+    return true
+  } else if (pnodnospatt.test(text) === true) {
+    return true
+  } else if (pslpatt.test(text) === true) {
+    return true
+  } else if (pnodpatt.test(text) === true) {
+    return true
+  } else if (pnospatt.test(text) === true) {
+    return true
+  } else {
+    return false
+  }
+};
+// WIP
+function findError (entry) {
+  var fixedEntry = entry
+  if (/\s{0,4}\*\s\[.*?\]\(http.*?\)/.test(entry) === false) { // Fix * site name/URL
+    console.log('marker2')
+    entry.replace(/^(\s*)\**\s*\[*([^()\[\]]*?)\]*\s*\(*(https*:\/\/[^ )]{1,})(\)*\s*`*⚠`*\s*-*\s*|\)*\s*-*\s*)(.*)/gi, function (match, p1, p2, p3, p4, p5) {
+      console.log('Error in [Name](URL)')
+      if (/⚠/.test(entry) === true) {
+        fixedEntry = p1 + '* [' + p2 + '](' + p3 + ') `⚠` - ' + p5
+      } else {
+        fixedEntry = p1 + '* [' + p2 + '](' + p3 + ') - ' + p5
+      }
+      console.log('Fixed: ' + fixedEntry)
+    })
+  }
+  if (/\s-\s/.test(entry) === false) {
+    console.log('marker3')
+    entry.replace(/(\s{0,3}\*\s\[.*?\]\(.*?\))(\s*`*⚠`*\s*-*\s*|\s*`*⚠`*\s*-*|`*⚠`*\s*-*\s*|`*⚠`*-*|\s*|\s*-*\s*|\s*-*|-*\s*|-*|\s*)(.{0,249}?\.)(.*)/gi, function (match, p1, p2, p3, p4) { // Fix space - space between link and descrition
+      console.log('Missing ` - ` between Site link and description.')
+      if (/⚠/.test(entry) === true) {
+        fixedEntry = p1 + ' `⚠` - ' + p3 + p4
+      } else {
+        fixedEntry = p1 + ' - ' + p3 + p4
+      }
+      console.log('Fixed: ' + fixedEntry)
+    })
+  }
+  if (/^\s{0,4}\*\s\[.*?\]\(.*?\).*?(\s\.|\)\)\s)`.*?`\s`.*?`/.test(entry) === false) {
+    console.log('marker4')
+    entry.replace(/^(\s*\*\s.*?\)\)|.*?\s\-\s.*?\.)(\s*`*\s*)(.*?)(\s*`\s*`\s*|\s*`\s*`*\s*|\s*`*\s*`\s*|\s)(.*?)`/gi, function (match, p1, p2, p3, p4, p5) { // Fix License and Language
+      console.log('Issue in Language or License')
+      fixedEntry = p1 + ' `' + p3 + '` `' + p5 + '`'
+      console.log('Fixed: ' + fixedEntry)
+    })
+  } else if (/demo/i.test(entry) === true) {
+
+  } else if (/source/i.test(entry) === true) {
+
+  }
+  console.log(findPattern(fixedEntry))
+  // if (findPattern(fixedEntry) === false) {
+    // findError(fixedEntry) // If it's not fixed let's try again.
+  // }
+}
+
+// else if (/source/i.test(entry) === true && /demo/i.test(entry) === true) {