[tahoe-lafs-trac-stream] [tahoe-lafs] #1992: warn if testing uncommitted code
tahoe-lafs
trac at tahoe-lafs.org
Wed May 29 21:52:11 UTC 2013
#1992: warn if testing uncommitted code
------------------------------------+------------------------------------
Reporter: daira | Owner: warner
Type: defect | Status: new
Priority: normal | Milestone: undecided
Component: dev-infrastructure | Version: 1.10.0
Resolution: | Keywords: test git review-needed
Launchpad Bug: |
------------------------------------+------------------------------------
Comment (by zooko):
Replying to [comment:4 gdt]:
> I'm not Brian, but I think the notion that one prepares a commit and
then tests it is sound. So the tests warning that the source tree is
dirty is a good idea.
Thanks, gdt! It sounds like a pretty good idea to me, too. I think a lot
of people get misdirected by git's terminology of calling a version a
"commit". There is no ''commitment'' involved. You can change your mind at
any time. ☺ Now a {{{git push}}}, to a repository that other people can
read, now that's commitment. Even then it might not be a big deal if you
are pushing to a repo+branch which everyone knows can get clobbered at any
time.
--
Ticket URL: <https://tahoe-lafs.org/trac/tahoe-lafs/ticket/1992#comment:5>
tahoe-lafs <https://tahoe-lafs.org>
secure decentralized storage
More information about the tahoe-lafs-trac-stream
mailing list