You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@camel.apache.org by zregvart <gi...@git.apache.org> on 2017/04/24 12:49:09 UTC

[GitHub] camel pull request #1656: CAMEL-11194 Upgrade Checkstyle and update rules

GitHub user zregvart opened a pull request:

    https://github.com/apache/camel/pull/1656

    CAMEL-11194 Upgrade Checkstyle and update rules

    This upgrades Checkstyle to the latest version currently available, refines the way license header in properties files are handled and sets the execution id of Checkstyle Maven plugin to `default-cli` so that the checks can be simply run with:
    
        mvn checkstyle:check
    
    Changes are split into multiple commits to make it easier to review.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/zregvart/camel CAMEL-11194

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/camel/pull/1656.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1656
    
----
commit c3bba7ae6d980448f1153c4f0c96755838c314e1
Author: Claus Ibsen <da...@apache.org>
Date:   2017-04-23T11:13:51Z

    Fixed license header

commit cf8670e7d643b3fe07776c974840ea0b78060093
Author: Jonathan Schoreels <jo...@gmail.com>
Date:   2017-04-23T13:52:22Z

    JdbcAggregationRepository : Adds some javadoc to clearly specify that properties are NOT persisted, and other properties as described in camel.apache.org

commit 3bf9847506cc0f963e3d5b0c6bc7774b0aab3f4e
Author: Jonathan Schoreels <jo...@gmail.com>
Date:   2017-04-23T14:07:53Z

    Adds JdbcAggregationRepository java doc to make clear that deadletter will be tried until it succeed

commit a4d2334ce7b95a964d1f029b5188818793118e8d
Author: Claus Ibsen <da...@apache.org>
Date:   2017-04-24T07:44:46Z

    Merge branch 'master' of https://github.com/JSchoreels/camel

commit 820410e2186a0135eab041de0f0e58843d21731c
Author: Claus Ibsen <da...@apache.org>
Date:   2017-04-24T08:17:29Z

    Skip connector examples as they are not intended for release.

commit 7a511ef5230b7447957e27a6082a6ac5e85e2f9f
Author: Chirag Anand <ca...@gp.cv.commvault.com>
Date:   2017-04-24T05:13:08Z

    Added the provision to pass Solr Params when pushing SolrInputDocuments.
    
    This change is necessary when you intend to have an update processor chain to be trigerred or any such for which parameters are to be passed along with the url.

commit d01d7777c3e345aad763db9a5e9c736df6727b65
Author: Claus Ibsen <da...@apache.org>
Date:   2017-04-24T08:30:53Z

    CAMEL-11185: Remove camel-scr-starter spring boot module

commit bab5b27bce8d9d799cfa4c77a1bfd1a1c023d281
Author: Claus Ibsen <da...@apache.org>
Date:   2017-04-24T08:46:47Z

    CAMEL-11189: Upgrade zipkin

commit 1d164d54675069fb672be606e99c2c7944cd8f23
Author: Claus Ibsen <da...@apache.org>
Date:   2017-04-24T11:03:56Z

    CAMEL-1193: Make kafka easier to use when routing between topics to avoid the header topic causing Camel to send the message to itself instead of the endpoint topic name as users would expect.

commit 8b5e93ec694ef31a2bd43248735d3bc274f4e9d3
Author: Claus Ibsen <da...@apache.org>
Date:   2017-04-24T11:24:20Z

    Make kafka bridgeEndpoint option on configuraion so they are all there and thus also available via spring boot

commit 6243402b2291af99c7f7b950f5e905489d403074
Author: Claus Ibsen <da...@apache.org>
Date:   2017-04-24T11:29:33Z

    Add some logging in kafka producer when it sends/has all sent etc.

commit 97ecc14735765749fdba823acf5e9ba1a5628424
Author: Claus Ibsen <da...@apache.org>
Date:   2017-04-24T11:40:33Z

    Logging level to info as debug is massiively verbose

commit 3ffd940a70994c9810b7b915d7a336fd8ad689cb
Author: Claus Ibsen <da...@apache.org>
Date:   2017-04-24T11:51:15Z

    CAMEL-11186: upgrade dropwizard metrics to align with Spring Boot

commit 27517e78e4ffcfafb502f1c863bc0d8aede871bc
Author: Zoran Regvart <zr...@apache.org>
Date:   2017-04-24T11:46:12Z

    CAMEL-11194 Upgrade Checkstyle and update rules
    
    This updates the Checkstyle version to 7.6.1, it also changes the
    execution id from `validate` to `default-cli` which allows the
    Checkstyle Maven plugin to be run with just `mvn checkstyle:check`.
    
    The change in Checkstyle rules now allows for different header comment
    on source files and on properties files. The rule that affects the use
    of `final` keyword and the rule that checks the `return` statement count
    per method were tweaked to reflect the current code base.

commit 13b0c17d865d15f06300f9cde317b1581b962c48
Author: Zoran Regvart <zr...@apache.org>
Date:   2017-04-24T11:45:21Z

    CAMEL-11194 Upgrade Checkstyle and update rules
    
    This updates the license header in properties files.

commit 5bfe72d54bb86deb8630985091aad540743178fa
Author: Zoran Regvart <zr...@apache.org>
Date:   2017-04-24T11:48:23Z

    CAMEL-11194 Upgrade Checkstyle and update rules
    
    Changes the formatting of Spring Boot auto configuration starters to use
    the built in formatter.
    
    The major change being the addition of newline at the end of the file.

commit aaf4ae9d78407ad520e92af1f2ed783ec1cffc0b
Author: Zoran Regvart <zr...@apache.org>
Date:   2017-04-24T11:48:50Z

    CAMEL-11194 Upgrade Checkstyle and update rules
    
    This fixes a number of Checkstyle issues mostly whitespace and `final`
    keyword use.

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] camel pull request #1656: CAMEL-11194 Upgrade Checkstyle and update rules

Posted by zregvart <gi...@git.apache.org>.
Github user zregvart closed the pull request at:

    https://github.com/apache/camel/pull/1656


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---