From commits-return-9792-archive-asf-public=cust-asf.ponee.io@fineract.apache.org Tue Jun 2 21:49:16 2020 Return-Path: X-Original-To: archive-asf-public@cust-asf.ponee.io Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [207.244.88.153]) by mx-eu-01.ponee.io (Postfix) with SMTP id 3485118064C for ; Tue, 2 Jun 2020 23:49:16 +0200 (CEST) Received: (qmail 1636 invoked by uid 500); 2 Jun 2020 21:49:15 -0000 Mailing-List: contact commits-help@fineract.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@fineract.apache.org Delivered-To: mailing list commits@fineract.apache.org Received: (qmail 1627 invoked by uid 99); 2 Jun 2020 21:49:15 -0000 Received: from ec2-52-202-80-70.compute-1.amazonaws.com (HELO gitbox.apache.org) (52.202.80.70) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 02 Jun 2020 21:49:15 +0000 From: =?utf-8?q?GitBox?= To: commits@fineract.apache.org Subject: =?utf-8?q?=5BGitHub=5D_=5Bfineract=5D_thesmallstar_commented_on_pull_request?= =?utf-8?q?_=23943=3A_FINERACT-1006_Added_spotless_to_auto_format_source_cod?= =?utf-8?q?e?= Message-ID: <159113455537.10999.15593529470792156071.asfpy@gitbox.apache.org> Date: Tue, 02 Jun 2020 21:49:15 -0000 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit In-Reply-To: References: thesmallstar commented on pull request #943: URL: https://github.com/apache/fineract/pull/943#issuecomment-637824763 For 2,3,4: I tried to see how many checkstyles can be automatically added through the generated XML file from your ".epf", it turned out space-related checkstyles were not automatically included. Then I found google-checks, this is just what google-java-formatter would do, and it happened to help automatically add the space-related check styles, I probably came to an incorrect conclusion of this being a bigger set and happen to use google-checks. If I am not wrong "epf" includes all preferences(also the formatting ), but XML only has formatting settings (which is required by formator). They are not interchangeable, we need to have them both. And spotless won't support the epf, it only takes XML as an argument hence I made it. google-checks vs our-old-xml -> Plan update, I will go around and stick to the generated XML from epf, and then add more tags as required to automate the checks from checkstyle. I was taking a very lazy approach earlier :P For 5,6,7 Good idea, working on it :) ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: users@infra.apache.org