From 3da769f6650283f864fd5dc1a27d77367d92cbd9 Mon Sep 17 00:00:00 2001 From: nrew225 Date: Sat, 10 Dec 2016 07:46:24 -0600 Subject: [PATCH] Initial Commit --- .travis.yml | 15 ++++++ Dangerfile | 36 ++++++++++++++ test.js | 132 ++++++++++++++++++++++++++++++++++++++++++++++++++++ 3 files changed, 183 insertions(+) create mode 100644 .travis.yml create mode 100644 Dangerfile create mode 100644 test.js diff --git a/.travis.yml b/.travis.yml new file mode 100644 index 00000000..fde40658 --- /dev/null +++ b/.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 diff --git a/Dangerfile b/Dangerfile new file mode 100644 index 00000000..8124c31b --- /dev/null +++ b/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 diff --git a/test.js b/test.js new file mode 100644 index 00000000..98d2ccd2 --- /dev/null +++ b/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) {