[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [O] [BUG] ob-sql.el: probably an extra paren
From: |
Sebastien Vauban |
Subject: |
Re: [O] [BUG] ob-sql.el: probably an extra paren |
Date: |
Thu, 21 Mar 2013 09:37:36 +0100 |
User-agent: |
Gnus/5.130006 (Ma Gnus v0.6) Emacs/24.3 (windows-nt) |
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
- Re: [O] tests with travis-ci, (continued)
- Re: [O] tests with travis-ci, Andreas Röhler, 2013/03/22
- Re: [O] [BUG] ob-sql.el: probably an extra paren, Achim Gratz, 2013/03/21
- Re: [O] [BUG] ob-sql.el: probably an extra paren, Bastien, 2013/03/21
- Re: [O] [BUG] ob-sql.el: probably an extra paren, Achim Gratz, 2013/03/22
- Re: [O] [BUG] ob-sql.el: probably an extra paren, Bastien, 2013/03/22
- Re: [O] [BUG] ob-sql.el: probably an extra paren, Achim Gratz, 2013/03/22
- Re: [O] [BUG] ob-sql.el: probably an extra paren, Yagnesh Raghava Yakkala, 2013/03/22
- Re: [O] [BUG] ob-sql.el: probably an extra paren,
Sebastien Vauban <=