[darcs-devel] [patch2144] refactor testing and running hooks

Ben Franksen bugs at darcs.net
Mon Dec 21 21:02:18 UTC 2020


New submission from Ben Franksen <ben.franksen at online.de>:

A common theme here is to move stuff that has nothing to do with Repository
mechanics, namely running hooks and tests, from the Repository to the UI
layer, especially since running hooks may involve prompting the user.

Another theme is to make the Repository and UI layers more functional and
less imperative, in this case by passing around a tree (the tentative
pristine) instead of a Repository.

8 patches for repository http://darcs.net/screened:

patch 954ba7d6aa1937d26ce534a83fd282d33e521461
Author: Ben Franksen <ben.franksen at online.de>
Date:   Thu Nov  5 13:50:44 CET 2020
  * bugfix: --leave-test-dir should be off by default

patch 9edd5f9f4d76bcc0c842ee721e64bf9111310876
Author: Ben Franksen <ben.franksen at online.de>
Date:   Thu Nov  5 14:08:45 CET 2020
  * support --leave-test-dir for all commands that support --test

patch 0cec9e038927db01b8ee2017ce6675c0defd874e
Author: Ben Franksen <ben.franksen at online.de>
Date:   Thu Nov  5 12:19:40 CET 2020
  * refactor testing of the tentative pristine

patch f4b961d97abf40d31f2c05ee35481eea5602e084
Author: Ben Franksen <ben.franksen at online.de>
Date:   Mon Dec 21 21:14:03 CET 2020
  * inline runTest into getTest and move to UI layer

patch fdf066a51e4bd051d43b6ed5498c5948477572eb
Author: Ben Franksen <ben.franksen at online.de>
Date:   Mon Dec 21 21:35:10 CET 2020
  * move remainder of D.R.Test to D.UI.RunHook

patch f27d28dc46a1fa8edcbaf6d73216cc9f50d5db63
Author: Ben Franksen <ben.franksen at online.de>
Date:   Mon Dec 21 21:37:24 CET 2020
  * reduce code duplication in Darcs.UI.RunHook

patch fc1d0a894334881117fb3553a9cb2afcba731b13
Author: Ben Franksen <ben.franksen at online.de>
Date:   Mon Dec 21 21:45:52 CET 2020
  * move Hook(s)Config from D.R.Flags to D.UI.Options.All

patch 52d8f1b62fecac9f93fcdfc4031d356f7b0d44c7
Author: Ben Franksen <ben.franksen at online.de>
Date:   Mon Dec 21 21:54:55 CET 2020
  * use promptYorn when asking whether to execute hook

----------
files: bugfix_-__leave_test_dir-should-be-off-by-default.dpatch, patch-preview.txt, unnamed
messages: 22589
nosy: bf
status: needs-screening
title: refactor testing and running hooks

__________________________________
Darcs bug tracker <bugs at darcs.net>
<http://bugs.darcs.net/patch2144>
__________________________________
-------------- next part --------------
A non-text attachment was scrubbed...
Name: patch-preview.txt
Type: text/x-darcs-patch
Size: 18371 bytes
Desc: not available
URL: <http://lists.osuosl.org/pipermail/darcs-devel/attachments/20201221/2a1a959b/attachment-0002.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: bugfix_-__leave_test_dir-should-be-off-by-default.dpatch
Type: application/x-darcs-patch
Size: 25640 bytes
Desc: not available
URL: <http://lists.osuosl.org/pipermail/darcs-devel/attachments/20201221/2a1a959b/attachment-0003.bin>
-------------- next part --------------
.






More information about the darcs-devel mailing list