Details

    • Type: Task Task
    • Status: Closed Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.0
    • Labels:
      None
    • Number of attachments :
      2

      Activity

      Hide
      Vincent Siveton added a comment -

      Dennis, I waited the faq to apply the patch...
      I committed, so feel free to work on it.

      Show
      Vincent Siveton added a comment - Dennis, I waited the faq to apply the patch... I committed, so feel free to work on it.
      Hide
      Dennis Lundberg added a comment -

      I've commited several changes to the documentation. My work is finished now and I've staged the new site here:
      http://people.apache.org/~dennisl/maven-verifier-plugin/

      We're still missing that FAQ though.

      Show
      Dennis Lundberg added a comment - I've commited several changes to the documentation. My work is finished now and I've staged the new site here: http://people.apache.org/~dennisl/maven-verifier-plugin/ We're still missing that FAQ though.
      Hide
      Denis Cabasson added a comment -

      We should change the usage.apt file, as it seems the maven-verifier-plugin is by default bound to the integration-test phase.
      So there is not really any point in declaring an executions/execution block in the build config.

      IMHO, let the plugin be bound to its default phase, is integration-test, rather than forcing its bind to the verify phase.

      (my wrong here, I didn't detect this when updating the documentation. I guess a patch isn't useful there )

      Show
      Denis Cabasson added a comment - We should change the usage.apt file, as it seems the maven-verifier-plugin is by default bound to the integration-test phase. So there is not really any point in declaring an executions/execution block in the build config. IMHO, let the plugin be bound to its default phase, is integration-test, rather than forcing its bind to the verify phase. (my wrong here, I didn't detect this when updating the documentation. I guess a patch isn't useful there )
      Hide
      Dennis Lundberg added a comment -

      Denis, should we remove the whole <executions> element?

      Show
      Dennis Lundberg added a comment - Denis, should we remove the whole <executions> element?
      Hide
      Dennis Lundberg added a comment -

      I've added a small FAQ now. The plugin now passes the docck:check. I'm going to close this issue. Please open another issue for any remaining stuff.

      Show
      Dennis Lundberg added a comment - I've added a small FAQ now. The plugin now passes the docck:check. I'm going to close this issue. Please open another issue for any remaining stuff.

        People

        • Assignee:
          Dennis Lundberg
          Reporter:
          Marvin King
        • Votes:
          0 Vote for this issue
          Watchers:
          1 Start watching this issue

          Dates

          • Created:
            Updated:
            Resolved:

            Time Tracking

            Estimated:
            Original Estimate - 17 hours
            17h
            Remaining:
            Remaining Estimate - 17 hours
            17h
            Logged:
            Time Spent - Not Specified
            Not Specified