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

Update all the IGB App POM files to use new plugin repository

        Details

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

          Description

          We recently deployed the bundle-markdown-encoder to this repository:

          https://nexus.bioviz.org/repository/maven-releases/

          Let's find out if we can use it in IGB Apps on: https://bitbucket.org/account/user/lorainelab/projects/IAQ

          • 23andMe SNP Converter
          • Command Socket
          • Geometric Mean Operator
          • Get Average Score
          • Merge Annotation Operator
          • ProtAnnot
          • Super Simple IGB App

          ("development apps" will be a separate ticket)

            Attachments

              Issue Links

              relates to

              New Feature - A new feature of the product, which has yet to be developed. IGBF-2042 Configure bundle markdown encoder to build and deploy to nexus

              • Major - Major loss of function.
              • Closed

                Activity

                Descending order - Click to sort in ascending order
                Hide
                Permalink
                svallapu Sai Charan Reddy Vallapureddy (Inactive) added a comment -

                Tested this in my local. Everything works as expected. Moving this to done.

                Show
                svallapu Sai Charan Reddy Vallapureddy (Inactive) added a comment - Tested this in my local. Everything works as expected. Moving this to done.
                Hide
                Permalink
                shamika Shamika Gajanan Kulkarni (Inactive) added a comment -

                Yes. I have fixed it for Command Socket app. For Get Average Score app, the changes will be in the respective ticket. Rest of the apps look fine. Thank you.

                Show
                shamika Shamika Gajanan Kulkarni (Inactive) added a comment - Yes. I have fixed it for Command Socket app. For Get Average Score app, the changes will be in the respective ticket. Rest of the apps look fine. Thank you.
                Hide
                Permalink
                ann.loraine Ann Loraine added a comment -

                All open PR's merged. Please test:
                Check that:

                • Each IGB App (not in development apps) has built and there is a fresh jar and xml file in Downloads folder for same.
                Show
                ann.loraine Ann Loraine added a comment - All open PR's merged. Please test: Check that: Each IGB App (not in development apps) has built and there is a fresh jar and xml file in Downloads folder for same.
                Hide
                Permalink
                ann.loraine Ann Loraine added a comment -

                Yes please. If you notice anything broken please fix.

                Show
                ann.loraine Ann Loraine added a comment - Yes please. If you notice anything broken please fix.
                Hide
                Permalink
                shamika Shamika Gajanan Kulkarni (Inactive) added a comment -

                PR submitted for all apps. Links are as follows:
                https://bitbucket.org/lorainelab/23andme-snp-converter/pull-requests/7/igbf-2045-update-pomxml-to-use-new-plugin/diff
                https://bitbucket.org/lorainelab/command-socket/pull-requests/13/igbf-2045-update-pomxml-to-use-new-plugin/diff
                https://bitbucket.org/lorainelab/geometric-mean-operator/pull-requests/8/igbf-2045-update-pomxml-to-use-new-plugin/diff
                https://bitbucket.org/lorainelab/get-average-score/pull-requests/4/igbf-2045-update-pomxml-to-use-new-plugin/diff
                https://bitbucket.org/lorainelab/merge-annotation-operator/pull-requests/10/igbf-2045-update-pomxml-to-use-new-plugin/diff
                https://bitbucket.org/lorainelab/protannot/pull-requests/8/igbf-2045-update-pomxml-to-use-new-plugin/diff
                https://bitbucket.org/lorainelab/super-simple-igb-app/pull-requests/7/igbf-2045-update-pomxml-to-use-new-plugin/diff

                I forgot to mention I noticed some tags were missing in pom.xml for Command Socket app and they have been added too. Please let me know if it is okay to add them in this ticket.

                Show
                shamika Shamika Gajanan Kulkarni (Inactive) added a comment - PR submitted for all apps. Links are as follows: https://bitbucket.org/lorainelab/23andme-snp-converter/pull-requests/7/igbf-2045-update-pomxml-to-use-new-plugin/diff https://bitbucket.org/lorainelab/command-socket/pull-requests/13/igbf-2045-update-pomxml-to-use-new-plugin/diff https://bitbucket.org/lorainelab/geometric-mean-operator/pull-requests/8/igbf-2045-update-pomxml-to-use-new-plugin/diff https://bitbucket.org/lorainelab/get-average-score/pull-requests/4/igbf-2045-update-pomxml-to-use-new-plugin/diff https://bitbucket.org/lorainelab/merge-annotation-operator/pull-requests/10/igbf-2045-update-pomxml-to-use-new-plugin/diff https://bitbucket.org/lorainelab/protannot/pull-requests/8/igbf-2045-update-pomxml-to-use-new-plugin/diff https://bitbucket.org/lorainelab/super-simple-igb-app/pull-requests/7/igbf-2045-update-pomxml-to-use-new-plugin/diff I forgot to mention I noticed some tags were missing in pom.xml for Command Socket app and they have been added too. Please let me know if it is okay to add them in this ticket.
                Hide
                Permalink
                ann.loraine Ann Loraine added a comment -

                Thanks - please proceed with PR for all.

                Show
                ann.loraine Ann Loraine added a comment - Thanks - please proceed with PR for all.
                Hide
                Permalink
                shamika Shamika Gajanan Kulkarni (Inactive) added a comment - - edited

                Yes, I have checked the build for all the above apps. I tried running the pipelines for all of them and the pipeline build was successful for each app. Also, I downloaded the repository.xml and verified that it contains the description. I have also tried installing each app on App Manager in IGB by using the downloads URL of the above repositories and the README.md file description was visible for each of them.
                Please let me know if anything else needs to be done too. Thank you.

                Show
                shamika Shamika Gajanan Kulkarni (Inactive) added a comment - - edited Yes, I have checked the build for all the above apps. I tried running the pipelines for all of them and the pipeline build was successful for each app. Also, I downloaded the repository.xml and verified that it contains the description. I have also tried installing each app on App Manager in IGB by using the downloads URL of the above repositories and the README.md file description was visible for each of them. Please let me know if anything else needs to be done too. Thank you.
                Hide
                Permalink
                ann.loraine Ann Loraine added a comment -

                Please check that the build for above changes has worked and that repository.xml contains the output from bundle-markdown-encoder.
                I looked in the downloads folder for 23 and me converter but the most recent files were dated Oct 4. So I'm wondering if the build is still working.

                Show
                ann.loraine Ann Loraine added a comment - Please check that the build for above changes has worked and that repository.xml contains the output from bundle-markdown-encoder. I looked in the downloads folder for 23 and me converter but the most recent files were dated Oct 4. So I'm wondering if the build is still working.
                Hide
                Permalink
                ann.loraine Ann Loraine added a comment - - edited

                I just looked at the first diff. It looks fine.

                Show
                ann.loraine Ann Loraine added a comment - - edited I just looked at the first diff. It looks fine.
                Hide
                Permalink
                shamika Shamika Gajanan Kulkarni (Inactive) added a comment -

                Kindly review the following changes for all apps:
                https://bitbucket.org/skulka2710/23andme-snp-converter-shamika/branch/IGBF-2045#diff
                https://bitbucket.org/skulka2710/command-socket-sgk-fork/branch/IGBF-2045#diff
                https://bitbucket.org/skulka2710/geometric-mean-operator-shamika/branch/IGBF-2045#diff
                https://bitbucket.org/skulka2710/get-average-score-shamika/branch/IGBF-2045#diff
                https://bitbucket.org/skulka2710/sgk-merge-annotation-operator/branch/IGBF-2045#diff
                https://bitbucket.org/skulka2710/protannot-shamika/branch/IGBF-2045#diff
                https://bitbucket.org/skulka2710/shamika-super-simple-igb-app/branch/IGBF-2045#diff

                Show
                shamika Shamika Gajanan Kulkarni (Inactive) added a comment - Kindly review the following changes for all apps: https://bitbucket.org/skulka2710/23andme-snp-converter-shamika/branch/IGBF-2045#diff https://bitbucket.org/skulka2710/command-socket-sgk-fork/branch/IGBF-2045#diff https://bitbucket.org/skulka2710/geometric-mean-operator-shamika/branch/IGBF-2045#diff https://bitbucket.org/skulka2710/get-average-score-shamika/branch/IGBF-2045#diff https://bitbucket.org/skulka2710/sgk-merge-annotation-operator/branch/IGBF-2045#diff https://bitbucket.org/skulka2710/protannot-shamika/branch/IGBF-2045#diff https://bitbucket.org/skulka2710/shamika-super-simple-igb-app/branch/IGBF-2045#diff

                  People

                  • Assignee:
                    shamika Shamika Gajanan Kulkarni (Inactive)
                    Reporter:
                    ann.loraine Ann Loraine
                  • Votes:
                    0 Vote for this issue
                    Watchers:
                    3 Start watching this issue

                    Dates

                    • Created:
                      Updated:
                      Resolved: