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

Remove NCBI Primer App code from experimental apps bitbucket repo

    Details

    • Type: Task
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: None
    • Labels:
      None
    • Story Points:
      0.25
    • Sprint:
      Spring 6 : 30 Mar to Apr 10, Spring 7 : 13 Apr to 24 Apr, Spring 8 : 24 Apr to 8 May

      Description

      Now that the NCBI Primer App is in its own repo, we need to remove it from the experimental apps bitbucket repository.

        Attachments

          Issue Links

            Activity

            Hide
            prutha Prutha Kulkarni (Inactive) added a comment -

            Removed the NCBI Primer from the experimental app repo and made changes in the super pom accordingly.
            Please refer the link mentioned below to check the same:
            https://bitbucket.org/pkulka10/experimental-apps-for-igb/src/IGBF-2328/

            Show
            prutha Prutha Kulkarni (Inactive) added a comment - Removed the NCBI Primer from the experimental app repo and made changes in the super pom accordingly. Please refer the link mentioned below to check the same: https://bitbucket.org/pkulka10/experimental-apps-for-igb/src/IGBF-2328/
            Hide
            ann.loraine Ann Loraine added a comment - - edited

            One change request:

            Please remove imports from the parent pom that were needed by NCBI primer app only. This includes apollo library and possibly others, as well.

            cc: Prutha Kulkarni

            Show
            ann.loraine Ann Loraine added a comment - - edited One change request: Please remove imports from the parent pom that were needed by NCBI primer app only. This includes apollo library and possibly others, as well. cc: Prutha Kulkarni
            Hide
            prutha Prutha Kulkarni (Inactive) added a comment -

            I have now removed the dependency which was specifically added to super pom for NCBI primer. Also, I have pushed everything into one commit so that it would be easy to figure that out in future.
            Changes are present at:
            https://bitbucket.org/pkulka10/experimental-apps-for-igb/commits/58ebaf917e4f2968fd1002a675804aadbf5dd86c

            Show
            prutha Prutha Kulkarni (Inactive) added a comment - I have now removed the dependency which was specifically added to super pom for NCBI primer. Also, I have pushed everything into one commit so that it would be easy to figure that out in future. Changes are present at: https://bitbucket.org/pkulka10/experimental-apps-for-igb/commits/58ebaf917e4f2968fd1002a675804aadbf5dd86c
            Hide
            ann.loraine Ann Loraine added a comment -

            My apologies. We are not yet ready to do this as code and branch for https://jira.transvar.org/browse/IGBF-2287 need to get merged into the master branch.

            At that point, we can move NCBI primer app into its own repository. After that is done, we can remove it from experimental apps bitbucket repo.

            Moving this back to "To-Do"

            cc: Pooja Nikhare and Prutha Kulkarni

            Show
            ann.loraine Ann Loraine added a comment - My apologies. We are not yet ready to do this as code and branch for https://jira.transvar.org/browse/IGBF-2287 need to get merged into the master branch. At that point, we can move NCBI primer app into its own repository. After that is done, we can remove it from experimental apps bitbucket repo. Moving this back to "To-Do" cc: Pooja Nikhare and Prutha Kulkarni
            Hide
            ann.loraine Ann Loraine added a comment - - edited
            Show
            ann.loraine Ann Loraine added a comment - - edited This now done. See: https://bitbucket.org/lorainelab/ncbi-primer

              People

              • Assignee:
                Unassigned
                Reporter:
                ann.loraine Ann Loraine
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: