Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [egit-dev] Commit Validation

What is about commit & push? The validation (and any amend) will be too late.
I'm wondering if this validation schould be added after/before the "pre-commit" hooks.
Also the planned commit/soft reset scenario in case of failed checks makes me nerveus.

Am 9. März 2015 07:11:53 MEZ, schrieb Duft Markus <Markus.Duft@xxxxxxxxxxxxxxxx>:
>Hey,
>
>As discussed with some of you the last 2 years of EclipseCon Europe, I
>now FINALLY have time to start working on a "commit validation"
>feature. It should be rather simple: After a commit has been done (1),
>the commit object is passed to a set of registered validators. They may
>object or not, returning an IStatus. A dialog is presented to the user,
>showing him a list of validation warnings/errors. In case there are
>errors, the commit soft-reset, and the commit message get's prefilled
>with the commits message again.
>
>
>(1)    /After/ the commit because it is way easier to have a look at a
>commit object and its contents than having to check the whole
>worktree... we already discussed this @ EclipseCon :)
>
>Does that sound like a good idea?
>
>I'd need a suggestion on how to register validators in EGit - should I
>create an extension point, or is there a better EntryPoint (register on
>repository?).
>
>Thanks for any feedback.
>
>Cheers,
>Markus
>
>--
>Mit freundlichen Grüßen / Best regards
>
>Markus Duft | Software Architect
>SSI SCHÄFER | Salomon Automation GmbH | Friesachstraße 15 | 8114
>Friesach bei Graz | Austria
>Phone +43 3127 200-575 | Fax +43 3127 200-22
>markus.duft@xxxxxxxxxxxxxxxx<mailto:markus.duft@xxxxxxxxxxxxxxxx>
>Website<http://www.salomon.at/> |
>Blog<http://www.ssi-schaefer.de/blog/> |
>YouTube<http://www.youtube.com/user/lagerlogistik1> |
>Facebook<https://www.facebook.com/SSI.SCHAEFER.DE>
>
>Salomon Automation GmbH | Friesachstrasse 15 | 8114 Friesach bei Graz |
>Austria
>Registered Office: Friesach bei Graz | Commercial Register: 49324 K |
>VAT no. ATU28654300
>Commercial Court: Landesgericht für Zivilrechtssachen Graz
>
>
>------------------------------------------------------------------------
>
>_______________________________________________
>egit-dev mailing list
>egit-dev@xxxxxxxxxxx
>To change your delivery options, retrieve your password, or unsubscribe
>from this list, visit
>https://dev.eclipse.org/mailman/listinfo/egit-dev

--
Kind regards,
Andrey Loskutov

http://google.com/+AndreyLoskutov


Back to the top