Sorry, that link should have been https://github.com/phillipuniverse/githook-maven-plugin. Anyway, I don't have experience with any particular plugin, but git hooks seem to be the obvious way to go and that's the first one I found.
-- Michael Mior [email protected] Le lun. 19 nov. 2018 à 15:36, Enrico Olivelli <[email protected]> a écrit : > Il lun 19 nov 2018, 21:29 Michael Mior <[email protected]> ha scritto: > > > How about making use of > https://github.com/olukyrich/githook-maven-plugin? > > A post-commit hooks in git seems to be an easy way to achieve this. > > Unfortunately, it would require that each fresh clone of the repository > has > > a one-time command run to install the hook. > > > > Michael, > It seems this plugin is notnot available on maven central > https://github.com/olukyrich/githook-maven-plugin > > Do you have experience with it? > Enrico > > > > > -- > > Michael Mior > > [email protected] > > > > > > Le lun. 19 nov. 2018 à 14:49, Julian Hyde <[email protected]> a écrit : > > > > > > > > > On Nov 19, 2018, at 11:19 AM, Vladimir Sitnikov < > > > [email protected]> wrote: > > > > > > > > Well, a rule of "first line should be separated by a blank line" > seems > > to > > > > be automatable. > > > > The rule of "CALCITE-XXX should be in [...]" seems to be automatable. > > > > And so on. > > > > > > Yes, we should do that. > > > > > > However there are things that automation could never achieve, so let’s > > > continue to talk about those, also. > > > > > > > setDynamicParam did not look good enough to me > > > > > > > > > > https://git-wip-us.apache.org/repos/asf?p=calcite.git;a=blobdiff;f=core/src/main/java/org/apache/calcite/runtime/ResultSetEnumerable.java;h=52c11f9aaf752cea367ae921c04a20e5e6da5488;hp=771772f1ea39d74eee6e9f889ae8d6500f129c7f;hb=53e15af6c5e8e782b2edcd7f5bf4f5f32225d110;hpb=02ca9bc995cac5b4b97855a4d06df46e632d7c22 > > > > > > > > I'm inclined to incline Avatica to expose > > > > TypedValue.setToPreparedStatement(PreparedStatement ps, int index) > kind > > > of > > > > API, so ResultSetEnumerable.setDynamicParam could be removed > > altogether. > > > > > > I agree. The commit didn’t seem quite perfect to me either. However, it > > > seemed to be progress. Log an Avatica JIRA if you have ideas for how to > > > improve it further. Since it will be in Avatica it will take a while to > > > bubble through the release cycle. > > > > > > Julian > > > > > > > > > -- > > > -- Enrico Olivelli >
