Uploaded image for project: 'IGB'
  1. IGB
  2. IGBF-2228

Build javadocs for Genoviz SDK and copy compiled javadocs to Downloads

    Details

    • Type: Task
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: None
    • Labels:
      None

      Description

      GenoViz SDK is a library of visualization components for building genome-browser applications.

      Currently we do not distribute compiled javadocs for the library. We should start doing this.

      For this task:

      • add code to the GenoViz POM to build the javadocs.
      • add code to the bitbucket pipelines yml file to copy the javadocs to the Downloads section of the repository

        Attachments

          Issue Links

            Activity

            Hide
            ann.loraine Ann Loraine added a comment -

            Was this moved to Needs Review by mistake?
            (There are no comments or notes explaining what was done.)

            Show
            ann.loraine Ann Loraine added a comment - Was this moved to Needs Review by mistake? (There are no comments or notes explaining what was done.)
            Hide
            stiwari8 Srishti Tiwari (Inactive) added a comment -

            Changes Summary:
            1. Corrected parent pom parameter as it was no longer compiling.
            2. Fixed problems in Javadocs like missing tags, syntax error, malformed HTML etc.
            3. Added command to build javadoc in default pipeline - ( mvn clean install javadoc:jar -D skipTests=false) and commands to copy javadoc jars to downloads directory

            Files changed:
            bitbucket-pipelines.yml
            genoviz_sdk/src/main/java/com/affymetrix/genoviz/bioviews/MapGlyphFactory.java
            genoviz_sdk/src/main/java/com/affymetrix/genoviz/glyph/AbstractResiduesGlyph.java
            genoviz_sdk/src/main/java/com/affymetrix/genoviz/glyph/AlignmentGlyph.java
            genoviz_sdk/src/main/java/com/affymetrix/genoviz/glyph/AxisGlyph.java
            genoviz_sdk/src/main/java/com/affymetrix/genoviz/glyph/ColorSepGlyph.java
            genoviz_sdk/src/main/java/com/affymetrix/genoviz/glyph/FlyPointLinkerGlyph.java
            genoviz_sdk/src/main/java/com/affymetrix/genoviz/parser/ContentParser.java
            genoviz_sdk/src/main/java/com/affymetrix/genoviz/widget/NeoAssembler.java
            pom.xml

            Please find the commit below for review:
            https://bitbucket.org/srishtitiwari/genoviz-sdk/commits/eb85e3467de50e2112cbd01323a60ab7f7da4e51

            Show
            stiwari8 Srishti Tiwari (Inactive) added a comment - Changes Summary: 1. Corrected parent pom parameter as it was no longer compiling. 2. Fixed problems in Javadocs like missing tags, syntax error, malformed HTML etc. 3. Added command to build javadoc in default pipeline - ( mvn clean install javadoc:jar -D skipTests=false) and commands to copy javadoc jars to downloads directory Files changed: bitbucket-pipelines.yml genoviz_sdk/src/main/java/com/affymetrix/genoviz/bioviews/MapGlyphFactory.java genoviz_sdk/src/main/java/com/affymetrix/genoviz/glyph/AbstractResiduesGlyph.java genoviz_sdk/src/main/java/com/affymetrix/genoviz/glyph/AlignmentGlyph.java genoviz_sdk/src/main/java/com/affymetrix/genoviz/glyph/AxisGlyph.java genoviz_sdk/src/main/java/com/affymetrix/genoviz/glyph/ColorSepGlyph.java genoviz_sdk/src/main/java/com/affymetrix/genoviz/glyph/FlyPointLinkerGlyph.java genoviz_sdk/src/main/java/com/affymetrix/genoviz/parser/ContentParser.java genoviz_sdk/src/main/java/com/affymetrix/genoviz/widget/NeoAssembler.java pom.xml Please find the commit below for review: https://bitbucket.org/srishtitiwari/genoviz-sdk/commits/eb85e3467de50e2112cbd01323a60ab7f7da4e51
            Hide
            ann.loraine Ann Loraine added a comment -

            Change request:

            • Add javadoc plugin to the POM.xml so that it will build the javadoc jar file as part of the default lifecycle
            • Check that when you run "mvn install" the javadocs get created and added to your local .m2 (for extra awesomeness, check that you can now navigate to the javadocs for GenoViz methods and classes.)
            • Change the mvn commands in the bitbucket pipelines yml file to "mvn install" (I'm pretty sure we don't need "clean")

            If we ever start developing GenoViz SDK, then we will move the javadoc plugin jar task into a separate profile (out of the default profile) to allow for building code without building the javadoc jar. But for now, we should compile the library and the javadoc jar files during the default lifecycle. Also, if we do this, then I can run the "deploy to nexus" custom pipeline and the javadocs will get released along with the compiled code and source code.

            Show
            ann.loraine Ann Loraine added a comment - Change request: Add javadoc plugin to the POM.xml so that it will build the javadoc jar file as part of the default lifecycle Check that when you run "mvn install" the javadocs get created and added to your local .m2 (for extra awesomeness, check that you can now navigate to the javadocs for GenoViz methods and classes.) Change the mvn commands in the bitbucket pipelines yml file to "mvn install" (I'm pretty sure we don't need "clean") If we ever start developing GenoViz SDK, then we will move the javadoc plugin jar task into a separate profile (out of the default profile) to allow for building code without building the javadoc jar. But for now, we should compile the library and the javadoc jar files during the default lifecycle. Also, if we do this, then I can run the "deploy to nexus" custom pipeline and the javadocs will get released along with the compiled code and source code.
            Hide
            ann.loraine Ann Loraine added a comment -

            Link showing how to configure javadoc plugin in a POM.xml:

            https://mkyong.com/maven/generate-javadoc-jar-for-maven-based-project/

            Show
            ann.loraine Ann Loraine added a comment - Link showing how to configure javadoc plugin in a POM.xml: https://mkyong.com/maven/generate-javadoc-jar-for-maven-based-project/
            Hide
            stiwari8 Srishti Tiwari (Inactive) added a comment -

            Hi Professor, I have made changes accordingly. Please have a look.

            https://bitbucket.org/srishtitiwari/genoviz-sdk/branch/IGBF-2228#diff

            Show
            stiwari8 Srishti Tiwari (Inactive) added a comment - Hi Professor, I have made changes accordingly. Please have a look. https://bitbucket.org/srishtitiwari/genoviz-sdk/branch/IGBF-2228#diff
            Hide
            ann.loraine Ann Loraine added a comment -

            Please check: Is the maven javadoc plugin already included in the super POM? If yes, you do not need to declare it in the Plugins section.

            Show
            ann.loraine Ann Loraine added a comment - Please check: Is the maven javadoc plugin already included in the super POM? If yes, you do not need to declare it in the Plugins section.
            Hide
            ann.loraine Ann Loraine added a comment -

            Please submit PR.

            Show
            ann.loraine Ann Loraine added a comment - Please submit PR.
            Show
            stiwari8 Srishti Tiwari (Inactive) added a comment - Pull Request Submitted: https://bitbucket.org/lorainelab/genoviz-sdk/pull-requests/18/igbf-2228/diff
            Hide
            pbole Pawan Bole (Inactive) added a comment -

            I am able to open the javadocs..

            Show
            pbole Pawan Bole (Inactive) added a comment - I am able to open the javadocs..

              People

              • Assignee:
                stiwari8 Srishti Tiwari (Inactive)
                Reporter:
                ann.loraine Ann Loraine
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: