Am 12.05.10 13:00, schrieb mmu: > Hi! > > We use the Subversion Integration via hook scripts and call > AgiloSVNPreCommit resp. AgiloSVNPostCommit from agilo.utils.svn_hooks. > Some errors are detected not until post-commit, e. g. the following: > “The task(#11) is not assigned (new), you have to accept it before > closing it!” > > Our first problem is that the error is not displayed in eclipse, not > even in SVN console view. > We are using Subclipse and eclipse 3.5.1. > > But the main problem is that the error should be detected already > during pre-commit! > If the commit is not aborted an inconsistent state will be the result: > The commited comment says “This bugfix closes #11.” but actually task > #11 remains open. > > Any ideas?
Well... on a first guess I'd say that this is probably one of the business rules and therefore the commit script doesn't check it before it tries to do it's magic. The thing with the business rules is that we don't really know which are active as we can't 'simulate' the ticket update in the precommit (if it worked it worked, and if the commit then fails, we changed the ticket anyway). Still this is a grave mistake... Would you be willing to give the pre_commit script a try and see if you can get it patched to check for this case? Have a look at agilo.utils.svn_hooks.py and agilo.utils.test.svn_hook_parsing_test.py if you want to give it a go. Regards, Martin -- -- Follow Agilo on Twitter: http://twitter.com/agiloforscrum Please support us by reviewing and voting on: http://userstories.com/products/8-agilo-for-scrum http://ohloh.net/p/agilo-scrum http://freshmeat.net/projects/agiloforscrum You received this message because you are subscribed to the Google Groups "Agilo for Scrum" group. This group is moderated by agile42 GmbH http://www.agile42.com and is focused in supporting Agilo for Scrum users. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [email protected] For more options, visit this group at http://groups.google.com/group/agilo?hl=en
