From 9df82dd2ce87e5b0e10b074a910c18372f9d6399 Mon Sep 17 00:00:00 2001 From: Chris Rebert Date: Fri, 10 Oct 2014 16:21:06 -0700 Subject: comment phrasing & formatting --- .../rorschach/auditing/BaseAndHeadBranchesAuditor.scala | 4 ++-- .../com/getbootstrap/rorschach/auditing/ModifiedFilesAuditor.scala | 6 +++--- 2 files changed, 5 insertions(+), 5 deletions(-) (limited to 'src') diff --git a/src/main/scala/com/getbootstrap/rorschach/auditing/BaseAndHeadBranchesAuditor.scala b/src/main/scala/com/getbootstrap/rorschach/auditing/BaseAndHeadBranchesAuditor.scala index 0122490..f412e78 100644 --- a/src/main/scala/com/getbootstrap/rorschach/auditing/BaseAndHeadBranchesAuditor.scala +++ b/src/main/scala/com/getbootstrap/rorschach/auditing/BaseAndHeadBranchesAuditor.scala @@ -10,14 +10,14 @@ object BaseAndHeadBranchesAuditor { def auditThatNotAgainstGhPages(baseBranch: String): Option[String] = { baseBranch match { - case "gh-pages" => Some("Normal pull requests should never be against the gh-pages branch.") + case "gh-pages" => Some("Normal pull requests should never be against the `gh-pages` branch.") case _ => None } } def auditThatNotMergingGhPagesIntoMaster(baseBranch: String, headBranch: String): Option[String] = { (headBranch, baseBranch) match { - case ("gh-pages", "master") => Some("Normally, gh-pages should not be merged into master.") + case ("gh-pages", "master") => Some("Normally, the `gh-pages` branch should never be merged into `master` branch.") case _ => None } } diff --git a/src/main/scala/com/getbootstrap/rorschach/auditing/ModifiedFilesAuditor.scala b/src/main/scala/com/getbootstrap/rorschach/auditing/ModifiedFilesAuditor.scala index 03dc9fd..67f3f3c 100644 --- a/src/main/scala/com/getbootstrap/rorschach/auditing/ModifiedFilesAuditor.scala +++ b/src/main/scala/com/getbootstrap/rorschach/auditing/ModifiedFilesAuditor.scala @@ -22,7 +22,7 @@ object ModifiedFilesAuditor { val cssModified = filepaths.exists{ _.isDistCss } val lessModified = filepaths.exists{ _.isSourceLess } if (cssModified && !lessModified) { - Some("Changes must be made to the original Less source code, not just the compiled CSS.") + Some("Changes must be made to the original Less source code file(s), not just the compiled CSS file(s).") } else { None @@ -36,7 +36,7 @@ object ModifiedFilesAuditor { val distJsModified = filepaths.exists{ _.isDistJs } val sourceJsModified = filepaths.exists{ _.isSourceJs } if (distJsModified && !sourceJsModified) { - Some("Changes must be made to the original JS source code, not just the generated concatenated JS.") + Some("Changes must be made to the original JS source code file(s), not just the generated concatenated JS file(s).") } else { None @@ -45,7 +45,7 @@ object ModifiedFilesAuditor { private def auditCname(filepaths: Set[String]): Option[String] = { if (filepaths.contains("CNAME")) { - Some("The CNAME file should never be modified (except in extremely unlikely circumstances).") + Some("The `CNAME` file should never be modified (except in extremely unlikely circumstances).") } else { None -- cgit v1.2.3