You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@calcite.apache.org by za...@apache.org on 2021/09/24 15:30:53 UTC

[calcite] 02/04: Site: Remove contributors name from commit summary

This is an automated email from the ASF dual-hosted git repository.

zabetak pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/calcite.git

commit c7b37644089d4c241ac75c9c9e854c04cc91e01c
Author: Stamatis Zampetakis <za...@gmail.com>
AuthorDate: Fri Sep 24 15:15:06 2021 +0200

    Site: Remove contributors name from commit summary
    
    See discussion in dev@ list[1].
    
    [1] https://lists.apache.org/thread.html/r0e198225f7912fa3d6eb910b9c271b2856cf156e87caadca5bf4cd8e%40%3Cdev.calcite.apache.org%3E
---
 site/_docs/howto.md   | 3 ---
 site/develop/index.md | 5 ++---
 2 files changed, 2 insertions(+), 6 deletions(-)

diff --git a/site/_docs/howto.md b/site/_docs/howto.md
index 70111c2..11edc39 100644
--- a/site/_docs/howto.md
+++ b/site/_docs/howto.md
@@ -596,9 +596,6 @@ If there are conflicts it is better to ask the contributor to take this step,
 otherwise it is preferred to do this manually since it saves time and also
 avoids unnecessary notification messages to many people on GitHub.
 
-If the contributor is not a committer, add their name in parentheses at the end
-of the first line of the commit message.
-
 If the merge is performed via command line (not through the GitHub web
 interface), make sure the message contains a line "Close apache/calcite#YYY",
 where YYY is the GitHub pull request identifier.
diff --git a/site/develop/index.md b/site/develop/index.md
index b99bf0e..29770cd 100644
--- a/site/develop/index.md
+++ b/site/develop/index.md
@@ -143,7 +143,7 @@ Commit your change to your branch, and use a comment that starts with
 the JIRA case number, like this:
 
 {% highlight text %}
-[CALCITE-345] AssertionError in RexToLixTranslator comparing to date literal (FirstName LastName)
+[CALCITE-345] AssertionError in RexToLixTranslator comparing to date literal
 {% endhighlight %}
 
 If your change had multiple commits, use `git rebase -i master` to
@@ -158,6 +158,7 @@ description of the change.
  * The message is often, but not always, the same as the JIRA subject.
 If the JIRA subject is not clear, change it (perhaps move the original
 subject to the description of the JIRA case, if it clarifies).
+ * Leave a single space character after the JIRA id.
  * Start with a capital letter.
  * Do not finish with a period.
  * Use imperative mood ("Add a handler ...") rather than past tense
@@ -168,8 +169,6 @@ the implementation ("Add handler for FileNotFound").
  * If you are fixing a bug, it is sufficient to describe the bug
  ("NullPointerException if user is unknown") and people will correctly
  surmise that the purpose of your change is to fix the bug.
- * If you are not a committer, add your name in parentheses at the end
- of the message.
 
 Then push your commit(s) to GitHub, and create a pull request from
 your branch to the calcite master branch. Update the JIRA case