Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Trivial
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.7
    • Component/s: packaging
    • Labels:
      None

      Description

      We should have a standard list of excludes for when we run RAT from the command line (other projects do this with the RAT ant or maven plugins). As far as I know there isn't a nice way to do that yet so I created RAT-99

      Assuming something like that is excepted, what I have so far is:
      .rat-excludes
      README.md
      .gitignore
      .git/.*
      .svn/.*
      target.*
      clients/target.*
      core/target.*
      contrib/target.*
      project/plugins/target.*
      boot.*
      TODO$
      expected.out$

      1. KAFKA-151.patch
        4 kB
        Neha Narkhede

        Activity

        Hide
        nehanarkhede Neha Narkhede added a comment -

        Modified the .rat-excludes that Chris uploaded here, to include READMEs, Makefile, .git, .svn and more.

        Added a run-rat.sh script that reads the .rat-excludes and pipes the rat output to kafka/rat.out

        Added a task to sbt that will run the above mentioned rat script.

        Added the rat jar to lib folder, since somehow it does't get pulled from Maven

        Show
        nehanarkhede Neha Narkhede added a comment - Modified the .rat-excludes that Chris uploaded here, to include READMEs, Makefile, .git, .svn and more. Added a run-rat.sh script that reads the .rat-excludes and pipes the rat output to kafka/rat.out Added a task to sbt that will run the above mentioned rat script. Added the rat jar to lib folder, since somehow it does't get pulled from Maven
        Hide
        junrao Jun Rao added a comment -

        +1

        Show
        junrao Jun Rao added a comment - +1
        Hide
        nehanarkhede Neha Narkhede added a comment -

        Kafka has a rat test that passes

        Show
        nehanarkhede Neha Narkhede added a comment - Kafka has a rat test that passes

          People

          • Assignee:
            nehanarkhede Neha Narkhede
            Reporter:
            cburroughs Chris Burroughs
          • Votes:
            1 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development