Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Blocker Blocker
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.1.0
    • Component/s: core
    • Labels:
      None

      Description

      We should generate a clean RAT report before doing a release. See http://incubator.apache.org/rat/apache-rat-plugin/usage.html

      1. WHIRR-20.patch
        0.8 kB
        Patrick Hunt

        Activity

        Tom White created issue -
        Patrick Hunt made changes -
        Field Original Value New Value
        Assignee Patrick Hunt [ phunt ]
        Hide
        Patrick Hunt added a comment -

        this patch adds support for running a RAT report: "mvn apache-rat:check". First you must install apache RAT from trunk (0.7-SNAPSHOT) to enable this plugin. at some point we might make it part of the build, but for now I think leaving it as a separate command is best (run at user discretion)

        Show
        Patrick Hunt added a comment - this patch adds support for running a RAT report: "mvn apache-rat:check". First you must install apache RAT from trunk (0.7-SNAPSHOT) to enable this plugin. at some point we might make it part of the build, but for now I think leaving it as a separate command is best (run at user discretion)
        Patrick Hunt made changes -
        Attachment WHIRR-20.patch [ 12446748 ]
        Patrick Hunt made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Hide
        Tom White added a comment -

        +1

        I ran it successfully. BTW is there any reason to use the snapshot build to RAT rather than the latest stable one? Not a problem, just wondering if there's something new in there.

        Show
        Tom White added a comment - +1 I ran it successfully. BTW is there any reason to use the snapshot build to RAT rather than the latest stable one? Not a problem, just wondering if there's something new in there.
        Hide
        Patrick Hunt added a comment -

        re .6 vs .7snapshot – on the rat incubator website there's no indication of any official releases and their documentation on "how to get started with rat" just says to checkout from svn... I guess I just followed that. perhaps we can stay with snapshot and if we run into issues we should switch.

        Show
        Patrick Hunt added a comment - re .6 vs .7snapshot – on the rat incubator website there's no indication of any official releases and their documentation on "how to get started with rat" just says to checkout from svn... I guess I just followed that. perhaps we can stay with snapshot and if we run into issues we should switch.
        Hide
        Tom White added a comment -

        I've just committed this. Thanks Patrick!

        Show
        Tom White added a comment - I've just committed this. Thanks Patrick!
        Tom White made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]

          People

          • Assignee:
            Patrick Hunt
            Reporter:
            Tom White
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development