summaryrefslogtreecommitdiff
path: root/CONTRIBUTING.md
diff options
context:
space:
mode:
authorJanek Bogucki <janekdb@gmail.com>2015-08-21 18:17:39 +0100
committerJanek Bogucki <janekdb@gmail.com>2015-08-21 18:17:39 +0100
commita7e24ce7ee96d850bc0a6e1d9cd897c0ecf2ca32 (patch)
treec651457798df30c58ce2b508f3dbdcd4896348c7 /CONTRIBUTING.md
parent36b0138a4abbc96d130da4c2d9c840d163db15d5 (diff)
downloadscala-a7e24ce7ee96d850bc0a6e1d9cd897c0ecf2ca32.tar.gz
scala-a7e24ce7ee96d850bc0a6e1d9cd897c0ecf2ca32.tar.bz2
scala-a7e24ce7ee96d850bc0a6e1d9cd897c0ecf2ca32.zip
Sync commit advice to README.md and fix Scabot URL
CONTRIBUTING.md offered different advice on commit subject line length to README.md. Align CONTRIBUTING to README. Fix Scabot repo URL
Diffstat (limited to 'CONTRIBUTING.md')
-rw-r--r--CONTRIBUTING.md4
1 files changed, 2 insertions, 2 deletions
diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md
index e6557d78dd..d01a71b9bd 100644
--- a/CONTRIBUTING.md
+++ b/CONTRIBUTING.md
@@ -86,7 +86,7 @@ A pull request should consist of commits with messages that clearly state what p
Commit logs should be stated in the active, present tense.
-A commit's subject should be 60 characters or less. Overall, think of
+A commit's subject should be 72 characters or less. Overall, think of
the first line of the commit as a description of the action performed
by the commit on the code base, so use the active voice and the
present tense. That also makes the commit subjects easy to reuse in
@@ -113,7 +113,7 @@ Our pull request bot, Scabot, automatically builds all the commits in a PR indiv
Click on the little x next to a commit sha to go to the overview of the PR validation job. To diagnose a failure, consult the console output of the job that failed.
-See the [scala-jenkins-infra repo](https://github.com/scala/scala-jenkins-infra) and [Scabot repo](https://github.com/) for full details on PR validation. One tip you should know is that commenting `/rebuild` on a PR asks validation to be run again on the same commits. This is only necessary when a spurious failure occurred.
+See the [scala-jenkins-infra repo](https://github.com/scala/scala-jenkins-infra) and [Scabot repo](https://github.com/scala/scabot) for full details on PR validation. One tip you should know is that commenting `/rebuild` on a PR asks validation to be run again on the same commits. This is only necessary when a spurious failure occurred.
### Pass code review