Suggested change to PreRepositoryHook

Tomas Bjerre May 20, 2017

My plugin is reporting findings from the commit messages when commits are pushed. It does not necessarily have to be findings that should reject the commit from being pushed. It can be information, or warnings, that should just be communicated.

https://github.com/tomasbjerre/simple-bitbucket-commit-checker/blob/master/src/main/java/se/bjurr/sbcc/SbccPreReceiveRepositoryHook.java#L98

If I understand correctly I should, from BBS 6.0, implement this class instead:

https://developer.atlassian.com/static/javadoc/bitbucket-server/latest/spi/reference/com/atlassian/bitbucket/hook/repository/PreRepositoryHook.html#preUpdate(com.atlassian.bitbucket.hook.repository.PreRepositoryHookContext

If I understand correctly, it can only report feedback with RepositoryHookResult and only when commits are rejected. It cannot report back a text message regarding valid accepted commits.

I would suggest that being changed. That was pretty much the only reason that I started working on this plugin.

A quick googling did not tell my how to make such a request!

1 answer

1 accepted

1 vote
Answer accepted
Julius Davies _bit-booster_com_
Rising Star
Rising Star
Rising Stars are recognized for providing high-quality answers to other users. Rising Stars receive a certificate of achievement and are on the path to becoming Community Leaders.
May 20, 2017

I haven't ported my own hook to the new BB 6 approach, but from clicking around the javadocs a little, I think you can write messages using the 2nd parameter "T request" of the preUpdate() call.    This "request" param will probably implement RepositoryHookRequest, in which case you can call request.getScmHookDetails().out().

Suggest an answer

Log in or Sign up to answer
TAGS
AUG Leaders

Atlassian Community Events