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

Clean up R6 commits, create pull request to main-JDK8

        Details

          Description

          Situation: The R6 branch needs to be merged with main-JDK8. There are a few commits that were reverted and can be squashed, to clean up the commit history prior to merging.

          Task: Clean up the R6 branch commits. Create pull request.

            Attachments

              Issue Links

              is blocked by

              Improvement - An improvement or enhancement to an existing feature or task. IGBF-3431 Update maven-bundle-plugin

              • Major - Major loss of function.
              • Closed

                Activity

                Ascending order - Click to sort in descending order
                nfreese Nowlan Freese created issue -
                nfreese Nowlan Freese made changes -
                Field Original Value New Value
                Epic Link IGBF-3411 [ 22434 ]
                nfreese Nowlan Freese made changes -
                Link This issue is blocked by IGBF-3431 [ IGBF-3431 ]
                nfreese Nowlan Freese made changes -
                Status To-Do [ 10305 ] In Progress [ 3 ]
                nfreese Nowlan Freese made changes -
                Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
                nfreese Nowlan Freese made changes -
                Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
                nfreese Nowlan Freese made changes -
                Status First Level Review in Progress [ 10301 ] Ready for Pull Request [ 10304 ]
                nfreese Nowlan Freese made changes -
                Status Ready for Pull Request [ 10304 ] Pull Request Submitted [ 10101 ]
                Hide
                Permalink
                nfreese Nowlan Freese added a comment -

                Commits on branch: https://bitbucket.org/nfreese/nowlanfork-igb/commits/branch/IGBF-3451

                I dropped 8 commits total (4 commits that were then each reverted) from the main-JDK8-r6 branch. These commits may cause confusion later as they were eventually reimplemented in a separate commit once they were working. So dropping the initial commit and its reversion makes the commit history less confusing and would make it easier for us moving forward to correctly identify the commit that made the correct change.

                I have then rebased the r6 changes onto main-JDK8 and tested through the installer. The pull request has been created from my branch, IGBF-3451 to main-JDK8, instead of from main-JDK8-r6 as the rebase on main-JDK8 would otherwise cause conflicts when merging to main-JDK-r6.

                Pull request: https://bitbucket.org/lorainelab/integrated-genome-browser/pull-requests/950

                Show
                nfreese Nowlan Freese added a comment - Commits on branch: https://bitbucket.org/nfreese/nowlanfork-igb/commits/branch/IGBF-3451 I dropped 8 commits total (4 commits that were then each reverted) from the main-JDK8-r6 branch. These commits may cause confusion later as they were eventually reimplemented in a separate commit once they were working. So dropping the initial commit and its reversion makes the commit history less confusing and would make it easier for us moving forward to correctly identify the commit that made the correct change. I have then rebased the r6 changes onto main-JDK8 and tested through the installer. The pull request has been created from my branch, IGBF-3451 to main-JDK8, instead of from main-JDK8-r6 as the rebase on main-JDK8 would otherwise cause conflicts when merging to main-JDK-r6. Pull request: https://bitbucket.org/lorainelab/integrated-genome-browser/pull-requests/950
                nfreese Nowlan Freese made changes -
                Assignee Nowlan Freese [ nfreese ]
                ann.loraine Ann Loraine made changes -
                Assignee Ann Loraine [ aloraine ]
                ann.loraine Ann Loraine made changes -
                Status Pull Request Submitted [ 10101 ] Reviewing Pull Request [ 10303 ]
                Hide
                Permalink
                ann.loraine Ann Loraine added a comment -

                Variable colon and hyphen usage in the commit messages warms my heart

                Show
                ann.loraine Ann Loraine added a comment - Variable colon and hyphen usage in the commit messages warms my heart
                Hide
                Permalink
                ann.loraine Ann Loraine added a comment -

                Merged.

                Show
                ann.loraine Ann Loraine added a comment - Merged.
                nfreese Nowlan Freese made changes -
                Status Reviewing Pull Request [ 10303 ] Merged Needs Testing [ 10002 ]
                nfreese Nowlan Freese made changes -
                Assignee Ann Loraine [ aloraine ]
                Hide
                Permalink
                nfreese Nowlan Freese added a comment -

                Tested using main-JDK8.dmg installer on Mac. IGB installed and ran correctly.

                Closing ticket.

                Show
                nfreese Nowlan Freese added a comment - Tested using main-JDK8.dmg installer on Mac. IGB installed and ran correctly. Closing ticket.
                nfreese Nowlan Freese made changes -
                Status Merged Needs Testing [ 10002 ] Post-merge Testing In Progress [ 10003 ]
                nfreese Nowlan Freese made changes -
                Resolution Done [ 10000 ]
                Status Post-merge Testing In Progress [ 10003 ] Closed [ 6 ]
                nfreese Nowlan Freese made changes -
                Assignee Nowlan Freese [ nfreese ]
                pkulzer Paige Kulzer made changes -
                Fix Version/s 10.0.0 Major Release [ 10900 ]

                  People

                  • Assignee:
                    nfreese Nowlan Freese
                    Reporter:
                    nfreese Nowlan Freese
                  • Votes:
                    0 Vote for this issue
                    Watchers:
                    2 Start watching this issue

                    Dates

                    • Created:
                      Updated:
                      Resolved: