Details

    • Type: Task Task
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.1
    • Labels:
      None

      Activity

      Hide
      Marvin King added a comment -

      for review

      Show
      Marvin King added a comment - for review
      Hide
      Allan Ramirez added a comment -

      Hi Marvin,

      The patch is not docck compliant

      here is the result when I checked it.

      The following documentation problems were found:

      o Maven EJB Plugin (5 errors, 0 warnings)
      [ERROR] Missing tag <prerequisites>
      [ERROR] Report plugin not found: maven-javadoc-plugin
      [ERROR] Report plugin not found: maven-jxr-plugin
      [ERROR] site.xml is missing link to: plugin-info.html "Goals"
      [ERROR] Missing base FAQ.(fml|html|xml|apt).

      Probably you missed to include your modified pom in your patch

      Show
      Allan Ramirez added a comment - Hi Marvin, The patch is not docck compliant here is the result when I checked it. The following documentation problems were found: o Maven EJB Plugin (5 errors, 0 warnings) [ERROR] Missing tag <prerequisites> [ERROR] Report plugin not found: maven-javadoc-plugin [ERROR] Report plugin not found: maven-jxr-plugin [ERROR] site.xml is missing link to: plugin-info.html "Goals" [ERROR] Missing base FAQ.(fml|html|xml|apt). Probably you missed to include your modified pom in your patch
      Hide
      Marvin King added a comment -


      documentation fix

      Show
      Marvin King added a comment - documentation fix
      Hide
      Marvin King added a comment -


      another documentation fix

      Show
      Marvin King added a comment - another documentation fix
      Hide
      Allan Ramirez added a comment -
      Show
      Allan Ramirez added a comment - I deployed your site here http://people.apache.org/~aramirez/maven-ejb-plugin/ Thanks
      Hide
      Allan Ramirez added a comment -

      Btw, please add your FAQ so I could apply your patch

      Show
      Allan Ramirez added a comment - Btw, please add your FAQ so I could apply your patch
      Hide
      Siveton Vincent added a comment -

      index.html
      Using section and subsection like our template [1]. In the same way, renaming:
      Introduction => Maven EJB Plugin
      Supported Goals => Goals Overview
      Add Usage part

      usage.html
      Introduction => Usage
      "Generating and customizing an ejb-client" link is broken

      [1] http://docs.codehaus.org/display/MAVEN/Maven+Plugin+Documentation

      Show
      Siveton Vincent added a comment - index.html Using section and subsection like our template [1] . In the same way, renaming: Introduction => Maven EJB Plugin Supported Goals => Goals Overview Add Usage part usage.html Introduction => Usage "Generating and customizing an ejb-client" link is broken [1] http://docs.codehaus.org/display/MAVEN/Maven+Plugin+Documentation
      Hide
      Marvin King added a comment -
      • documentation fix to comply with the standard plugin documentation
      Show
      Marvin King added a comment - documentation fix to comply with the standard plugin documentation
      Hide
      Anagnostopoulos Kostis added a comment -

      Please merge this patch ASAP, the info contained in "Specifying an ejb client as dependency" is priceless!

      Thank you in advance.

      Show
      Anagnostopoulos Kostis added a comment - Please merge this patch ASAP, the info contained in "Specifying an ejb client as dependency" is priceless! Thank you in advance.
      Hide
      Dennis Lundberg added a comment -

      The patch has been applied with some modifications.
      An FAQ has been added.
      The site has been deployed.

      Show
      Dennis Lundberg added a comment - The patch has been applied with some modifications. An FAQ has been added. The site has been deployed.

        People

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

          Dates

          • Due:
            Created:
            Updated:
            Resolved:

            Time Tracking

            Estimated:
            Original Estimate - 14h
            14h
            Remaining:
            Time Spent - 13h Remaining Estimate - 1h
            1h
            Logged:
            Time Spent - 13h Remaining Estimate - 1h
            13h

              Development