]> git.vanrenterghem.biz Git - git.ikiwiki.info.git/blobdiff - doc/todo/Add_instructive_commit_messages_for_removing_pages.mdwn
Add missing comma.
[git.ikiwiki.info.git] / doc / todo / Add_instructive_commit_messages_for_removing_pages.mdwn
index f000878b751069f027c7300a38e01d8e338191ac..8b1dd74a7df832f38507b197512bf795668518a9 100644 (file)
@@ -2,6 +2,16 @@ As [[Add instructive commit messages for add _47_ edit pages]], but for `remove.
 
 I use a `join()` since it at least looks like the plugin is able to remove several pages at once (`foreach` looping over file parameters), thus holding multiple entries in `@pages`. I haven't seen this happen, though.
 
+> I feel that anything that shows a change should show what files were
+> changed (at least as an easily accessible option), so mentioning
+> filenames in commits is almost always clutter.
+> 
+> It could be argued that there should be no message at all here, unless
+> the user provides one (which they currently cannot), as is done when
+> adding files. But the entire removal of a page from a wiki is a fairly
+> unusual circumstance that is probably best highlighted as such in
+> recentchanges. --[[Joey]]
+
 Diff follows. --[[Daniel Andersson]]
 
 [[!tag patch]]