Hi Suraj, I'm currently writing a comment at OFBIZ-11304 (since this morning, with details)
With Aditya's work we have already installed a pre-push automated style check hook in trunk. So everybody using trunk HEAD will pass by it (except if using --no-verify option or removing the hook by hand). But in case of PR merge this has no effect (the related commit was a PR merge). Because the hook is only local. And Infra told me they can't install a git checkstyle pre receive hook because of GH. I'll complete all the details soon... Thanks to care :) Jacques Le 16/06/2020 à 15:40, Suraj Khurana a écrit :
Hello team, As Jacques confirms in the very next commit, this was due to a checkstyle issue introduced. I would like to request all contributors to please run './gradlew check' before creating PR's. TIA. -- Best Regards, Suraj Khurana Senior Technical Consultant On Tue, Jun 16, 2020 at 4:38 PM <[email protected]> wrote:The Buildbot has detected a new failure on builder ofbizTrunkFrameworkPlugins while building ofbiz-framework. Full details are available at: https://ci.apache.org/builders/ofbizTrunkFrameworkPlugins/builds/1494 Buildbot URL: https://ci.apache.org/ Buildslave for this Build: asf946_ubuntu Build Reason: downstream Build Source Stamp: [branch trunk] a3059c097dfd2f2ce85ba6eb90ac41f2556b1a91 Blamelist: Priya Sharma <[email protected]> BUILD FAILED: failed check Sincerely, -The Buildbot
