From: "Sebastien Vauban" <wxhgmqzgwmuf-geNee64TY+gS+FvcfC7Uqw@public.gmane.org>
To: emacs-orgmode-mXXj517/zsQ@public.gmane.org
Subject: Re: [BUG] ob-sql.el: probably an extra paren
Date: Thu, 21 Mar 2013 09:37:36 +0100 [thread overview]
Message-ID: <86ehf9p2r3.fsf@somewhere.org> (raw)
In-Reply-To: 87fvzqnpfd.fsf@bzg.ath.cx
Hi Bastien,
Bastien wrote:
> "Sebastien Vauban" writes:
>> Nick Dokos wrote:
>>> Can we please make it an invariable practice to run `make test' before
>>> every push?
>>
>> Isn't it possible to put such in some sort of Git pre-commit hook (or
>> pre-push hook), so that it gets automatically enforced?
>
> If anyone knows how to setup an automated tests framework for Org, feel free
> to go ahead, we will use it and monitor broken tests to see what's wrong in
> the code or in the tests or in the environment running the tests.
>
> Testing is a nice habit to have, but let's not make it a coercive
> pre-requisit before pushing patches.
>
> My whole thinking here is well captured by Rich Hickey:
>
> http://codequarterly.com/2011/rich-hickey/
>
> Fogus: You have been known to speak out against test-driven development.
> Do you mind elaborating on your position?
>
> Hickey: I never spoke out ‘against’ TDD. What I have said is, life is
> short and there are only a finite number of hours in a day. So, we have to
> make choices about how we spend our time. If we spend it writing tests,
> that is time we are not spending doing something else. Each of us needs to
> assess how best to spend our time in order to maximize our results, both
> in quantity and quality. If people think that spending fifty percent of
> their time writing tests maximizes their results—okay for them. I’m sure
> that’s not true for me—I’d rather spend that time thinking about my
> problem. I’m certain that, for me, this produces better solutions, with
> fewer defects, than any other use of my time. A bad design with a complete
> test suite is still a bad design.
The text you mention refers about time to write extra test suites. I was
referring to simply have "make test" _run_ before being able to push commits.
Best regards,
Seb
--
Sebastien Vauban
prev parent reply other threads:[~2013-03-21 8:37 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-03-20 2:54 [BUG] ob-sql.el: probably an extra paren Nick Dokos
2013-03-20 6:47 ` Bastien
2013-03-20 12:39 ` Sebastien Vauban
2013-03-20 13:47 ` Bastien
2013-03-21 8:28 ` Achim Gratz
2013-03-21 13:41 ` Bastien
2013-03-21 14:03 ` Nicolas Richard
2013-03-21 16:58 ` Yagnesh Raghava Yakkala
2013-03-21 17:31 ` Bastien
2013-03-21 18:40 ` tests with travis-ci (was: [BUG] ob-sql.el: probably an extra paren) Yagnesh Raghava Yakkala
2013-03-21 20:41 ` tests with travis-ci Bastien
2013-03-21 22:20 ` Yagnesh Raghava Yakkala
2013-03-22 7:56 ` Yann Hodique
2013-03-22 10:40 ` Yagnesh Raghava Yakkala
2013-03-31 19:07 ` Yann Hodique
2013-03-22 6:40 ` Andreas Röhler
2013-03-22 12:55 ` Nick Dokos
2013-03-22 13:07 ` Andreas Röhler
2013-03-21 17:38 ` [BUG] ob-sql.el: probably an extra paren Achim Gratz
2013-03-21 17:59 ` Bastien
2013-03-22 7:01 ` Achim Gratz
2013-03-22 7:36 ` Bastien
2013-03-22 7:59 ` Achim Gratz
2013-04-06 0:46 ` Bastien
2013-03-22 10:44 ` Yagnesh Raghava Yakkala
2013-03-21 8:37 ` Sebastien Vauban [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://www.orgmode.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=86ehf9p2r3.fsf@somewhere.org \
--to=wxhgmqzgwmuf-genee64ty+gs+fvcfc7uqw@public.gmane.org \
--cc=emacs-orgmode-mXXj517/zsQ@public.gmane.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).