Re: [PATCH] emacs: add notmuch-hello-refresh-hook

Subject: Re: [PATCH] emacs: add notmuch-hello-refresh-hook

Date: Wed, 21 Dec 2011 05:33:26 +0400

To: Thomas Jost

Cc: notmuch@notmuchmail.org

From: Dmitry Kurochkin

Hi Thomas.

Looks good to me.

We should also add tests for this, similar to those for
`notmuch-hello-mode-hook'.  Thomas, do you think you can work on it?

Regards,
  Dmitry

• Previous message (by thread): Re: [PATCH v6] emacs: Let the user choose where to compose new mails

Thread:

• Thomas Jost—[PATCH 0/4] Several minor Emacs enhancements [inbox, unread]
  • Thomas Jost—[PATCH 1/4] emacs: Add a face for crypto parts headers [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
  • Thomas Jost—[PATCH 2/4] emacs: Support a message-mode switch function in notmuch-mua [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
    • Jameson Graef Rollins—Re: [PATCH 2/4] emacs: Support a message-mode switch function in notmuch-mua [inbox, notmuch::review, signed, unread]
      • Jameson Graef Rollins—Re: [PATCH 2/4] emacs: Support a message-mode switch function in notmuch-mua [inbox, signed, unread]
      • Thomas Jost—Re: [PATCH 2/4] emacs: Support a message-mode switch function in notmuch-mua [inbox, signed, unread]
        • Thomas Jost—[PATCH v2] emacs: Support a message-mode switch function in notmuch-mua [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
          • Jani Nikula—Re: [PATCH v2] emacs: Support a message-mode switch function in notmuch-mua [inbox, unread]
        • Jameson Graef Rollins—Re: [PATCH 2/4] emacs: Support a message-mode switch function in notmuch-mua [inbox, signed, unread]
          • Thomas Jost—Re: [PATCH 2/4] emacs: Support a message-mode switch function in notmuch-mua [inbox, signed, unread]
            • Thomas Jost—[PATCH] emacs: Let the user choose where to compose new mails [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
              • Austin Clements—Re: [PATCH] emacs: Let the user choose where to compose new mails [inbox, notmuch::review, unread]
                • Thomas Jost—Re: [PATCH] emacs: Let the user choose where to compose new mails [inbox, signed, unread]
                  • Thomas Jost—[PATCH v2] emacs: Let the user choose where to compose new mails [inbox, notmuch::obsolete, notmuch::patch, unread]
                    • Jameson Graef Rollins—Re: [PATCH v2] emacs: Let the user choose where to compose new mails [inbox, notmuch::review, signed, unread]
  • Thomas Jost—[PATCH 3/4] emacs: rename notmuch-decimal-separator to notmuch-thousands-separator [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
  • Thomas Jost—[PATCH 4/4] emacs: add notmuch-hello-hook [inbox, notmuch::emacs, notmuch::obsolete, notmuch::patch, unread]
  • David Bremner—Re: [PATCH 0/4] Several minor Emacs enhancements [inbox, unread]
    • Thomas Jost—[PATCH v3 0/4] Several minor Emacs enhancements [inbox, unread]
      • Thomas Jost—[PATCH v3 1/4] emacs: Let the user choose where to compose new mails [inbox, notmuch::obsolete, notmuch::patch, unread]
        • David Bremner—Re: [PATCH v3 1/4] emacs: Let the user choose where to compose new mails [inbox, unread]
          • Jameson Graef Rollins—Re: [PATCH v3 1/4] emacs: Let the user choose where to compose new mails [inbox, signed, unread]
            • David Bremner—Re: [PATCH v3 1/4] emacs: Let the user choose where to compose new mails [inbox, unread]
              • Jameson Graef Rollins—Re: [PATCH v3 1/4] emacs: Let the user choose where to compose new mails [inbox, signed, unread]
                • David Bremner—Re: [PATCH v3 1/4] emacs: Let the user choose where to compose new mails [inbox, unread]
                  • Jameson Graef Rollins—Re: [PATCH v3 1/4] emacs: Let the user choose where to compose new mails [inbox, signed, unread]
                  • Tom Prince—Re: [PATCH v3 1/4] emacs: Let the user choose where to compose new mails [inbox, unread]
              • Aaron Ecay—Re: [PATCH v3 1/4] emacs: Let the user choose where to compose new mails [inbox, unread]
                • David Bremner—Re: [PATCH v3 1/4] emacs: Let the user choose where to compose new mails [attachment, inbox, unread]
                • Thomas Jost—Re: [PATCH v3 1/4] emacs: Let the user choose where to compose new mails [inbox, signed, unread]
        • Jameson Graef Rollins—Re: [PATCH v3 1/4] emacs: Let the user choose where to compose new mails [inbox, signed, unread]
          • Jameson Graef Rollins—[PATCH] emacs: Let the user choose where to compose new mails [inbox, notmuch::obsolete, notmuch::patch, unread]
            • Jameson Graef Rollins—[PATCH v5] emacs: Let the user choose where to compose new mails [inbox, unread]
            • Tomi Ollila—Re: [PATCH] emacs: Let the user choose where to compose new mails [inbox, notmuch::review, unread]
          • David Bremner—Re: [PATCH v3 1/4] emacs: Let the user choose where to compose new mails [inbox, unread]
            • Thomas Jost—Re: [PATCH v3 1/4] emacs: Let the user choose where to compose new mails [inbox, signed, unread]
              • David Bremner—Re: [PATCH v3 1/4] emacs: Let the user choose where to compose new mails [inbox, unread]
                • Thomas Jost—[PATCH v6] emacs: Let the user choose where to compose new mails [inbox, notmuch::patch, notmuch::pushed, unread]
                  • Jameson Graef Rollins—Re: [PATCH v6] emacs: Let the user choose where to compose new mails [inbox, signed, unread]
                  • David Bremner—Re: [PATCH v6] emacs: Let the user choose where to compose new mails [inbox, unread]
      • Thomas Jost—[PATCH v3 2/4] emacs: Add a face for crypto parts headers [inbox, notmuch::patch, notmuch::pushed, notmuch::trivial, unread]
        • Dmitry Kurochkin—Re: [PATCH v3 2/4] emacs: Add a face for crypto parts headers [inbox, unread]
        • David Bremner—Re: [PATCH v3 2/4] emacs: Add a face for crypto parts headers [inbox, unread]
      • Thomas Jost—[PATCH v3 3/4] emacs: rename notmuch-decimal-separator to notmuch-thousands-separator [inbox, notmuch::obsolete, notmuch::patch, unread]
        • Dmitry Kurochkin—Re: [PATCH v3 3/4] emacs: rename notmuch-decimal-separator to notmuch-thousands-separator [inbox, unread]
          • David Bremner—Re: [PATCH v3 3/4] emacs: rename notmuch-decimal-separator to notmuch-thousands-separator [inbox, unread]
            • Dmitry Kurochkin—Re: [PATCH v3 3/4] emacs: rename notmuch-decimal-separator to notmuch-thousands-separator [inbox, unread]
              • Thomas Jost—Re: [PATCH v3 3/4] emacs: rename notmuch-decimal-separator to notmuch-thousands-separator [inbox, signed, unread]
                • Dmitry Kurochkin—Re: [PATCH v3 3/4] emacs: rename notmuch-decimal-separator to notmuch-thousands-separator [inbox, unread]
                  • Thomas Jost—[PATCH 1/2] emacs: rename notmuch-decimal-separator to notmuch-hello-thousands-separator [inbox, notmuch::emacs, notmuch::patch, notmuch::pushed, notmuch::trivial, unread]
                    • Thomas Jost—[PATCH 2/2] emacs: Change the default thousands separator to a space [inbox, notmuch::emacs, notmuch::patch, notmuch::pushed, notmuch::trivial, unread]
                      • Tomi Ollila—Re: [PATCH 2/2] emacs: Change the default thousands separator to a space [inbox, notmuch::review, unread]
                    • Dmitry Kurochkin—Re: [PATCH 1/2] emacs: rename notmuch-decimal-separator to notmuch-hello-thousands-separator [inbox, notmuch::review, unread]
                    • David Bremner—Re: [PATCH 1/2] emacs: rename notmuch-decimal-separator to notmuch-hello-thousands-separator [inbox, unread]
      • Thomas Jost—[PATCH v3 4/4] emacs: add notmuch-hello-hook [inbox, notmuch::obsolete, notmuch::patch, unread]
        • Dmitry Kurochkin—Re: [PATCH v3 4/4] emacs: add notmuch-hello-hook [inbox, unread]
          • Dmitry Kurochkin—Re: [PATCH v3 4/4] emacs: add notmuch-hello-hook [inbox, unread]
            • Thomas Jost—[PATCH] emacs: add notmuch-hello-refresh-hook [inbox, notmuch::obsolete, notmuch::patch, unread]
              • Thomas Jost—[PATCH] emacs: add notmuch-hello-refresh-hook [inbox, notmuch::patch, notmuch::pushed, unread]
                • David Bremner—Re: [PATCH] emacs: add notmuch-hello-refresh-hook [inbox, unread]
                • David Bremner—Re: [PATCH] emacs: add notmuch-hello-refresh-hook [inbox, unread]
              • Dmitry Kurochkin—Re: [PATCH] emacs: add notmuch-hello-refresh-hook [inbox, notmuch::review, unread]
                • Thomas Jost—[PATCH] test: add tests for `notmuch-hello-refresh-hook' [inbox, notmuch::obsolete, notmuch::patch, unread]
                  • Dmitry Kurochkin—Re: [PATCH] test: add tests for `notmuch-hello-refresh-hook' [inbox, unread]
      • Jameson Graef Rollins—Re: [PATCH v3 0/4] Several minor Emacs enhancements [inbox, notmuch::review, signed, unread]