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

CLONE - On-boarding: Step 4 - add your name to Credits

        Details

        • Type: Task
        • Status: Closed (View Workflow)
        • Priority: Major
        • Resolution: Done
        • Affects Version/s: None
        • Fix Version/s: None
        • Labels:
        • Story Points:
          2
        • Epic Link:
        • Sprint:
          Fall 2018 Sprint 5, Winter 2018 Sprint 1, Winter 2018 Sprint 3, Spring 2019 Sprint 1, Spring 2019 Sprint 2, Summer 2019 Sprint 7, Summer 2019 Sprint 8, Summer 2019 Sprint 9, Fall 5 : 14 Oct to 25 Oct, Fall 6 : 28 Oct to 8 Nov, Fall 7 : 11 Nov to 22 Nov, Fall 8 : 25 Nov to 6 Dec, Spring 9 : 25 May to 8 Jun, Summer 3: 6 Jul - 17 Jul, Summer 4: 14 Jul - 28 Jul, Summer 6: 17 Aug - 28 Aug, Summer 7: 31 Aug - 11 Sep, Fall 3: Oct 12 - Oct 23, Winter 5 Feb 22 - Mar 5, Winter 6 Mar 8 - Mar 19, Spring 4 2021 May 3 - May 14, Spring 5 2021 May 17 - May 28, Summer 2 2023 May 29, Fall 4 2023 Oct 16, Spring 2, Spring 5, Fall 1, Fall 5, Fall 6, Fall 7, Winter 1

          Description

          The main goal of this ticket/issue is for you to practice the IGB workflow using Jira.

          You'll make a small change to the code base, test it, and then move your ticket forward in the board.

          To start, download and run IGB. Then, Select Help > About Integrated Genome Browser.

          Note there is a list of contributors. For this task, add your name to the list.

          Note:

          • Assign this issue to yourself, and move it into the current spring to the "In Progress" lane.
          • Read and review the comments and previous work done by others. Review the comments and history for the linked issues, as well.
          • Work on editing the proper section of code and adding your name to the credits. Use the IGB fork-and-branch workflow.
          • Once you've completed the code changes, run a successful pipeline, and commented a link to your branch, moved this ticket to the "Needs first level review" column, and unassign yourself.

          Reminders and tips:

          • Make sure your installer has built (as seen in your downloads section). Whoever does the functional review will download and run your installer.
          • Make sure you add a link to your fork and branch as a comment to the issue. Whoever does code review will need the link to find your fork and view your code.
          • Use correct voice and format for commit messages.

          Reviewers:

          • Check that the installer has built.
          • Check that all commit messages contain the ticket number.
          • Check that commit messages are imperative voice, present tense.
          • Check that the ticket contains a link to the fork.
          • Check that only one line of code is changed (no whitespace edits).

            Attachments

              Issue Links

              clones

              Task - A task that needs to be done. IGBF-1464 On-boarding: Step 4 - add your name to Credits

              • Major - Major loss of function.
              • Closed

                Activity

                Ascending order - Click to sort in descending order
                aloraine Ann Loraine created issue -
                aloraine Ann Loraine made changes -
                Field Original Value New Value
                Epic Link IGBF-1762 [ 17852 ]
                aloraine Ann Loraine made changes -
                Link This issue clones IGBF-1464 [ IGBF-1464 ]
                aloraine Ann Loraine made changes -
                Assignee Pranav Bhatia [ pbhatia1 ]
                aloraine Ann Loraine made changes -
                Sprint Fall 2018 Sprint 5, Winter 2018 Sprint 1, Winter 2018 Sprint 3, Spring 2019 Sprint 1, Spring 2019 Sprint 2, Summer 2019 Sprint 7, Summer 2019 Sprint 8, Summer 2019 Sprint 9, Fall 5 : 14 Oct to 25 Oct, Fall 6 : 28 Oct to 8 Nov, Fall 7 : 11 Nov to 22 Nov, Fall 8 : 25 Nov to 6 Dec, Spring 9 : 25 May to 8 Jun, Summer 3: 6 Jul - 17 Jul, Summer 4: 14 Jul - 28 Jul, Summer 6: 17 Aug - 28 Aug, Summer 7: 31 Aug - 11 Sep, Fall 3: Oct 12 - Oct 23, Winter 5 Feb 22 - Mar 5, Winter 6 Mar 8 - Mar 19, Spring 4 2021 May 3 - May 14, Spring 5 2021 May 17 - May 28, Summer 2 2023 May 29, Fall 4 2023 Oct 16, Spring 2, Spring 5, Fall 1, Fall 5, Fall 6, Fall 7 [ 55, 56, 58, 59, 60, 66, 67, 68, 76, 77, 78, 79, 95, 98, 99, 101, 102, 106, 115, 116, 120, 121, 171, 180, 186, 189, 202, 206, 207, 208 ] Fall 2018 Sprint 5, Winter 2018 Sprint 1, Winter 2018 Sprint 3, Spring 2019 Sprint 1, Spring 2019 Sprint 2, Summer 2019 Sprint 7, Summer 2019 Sprint 8, Summer 2019 Sprint 9, Fall 5 : 14 Oct to 25 Oct, Fall 6 : 28 Oct to 8 Nov, Fall 7 : 11 Nov to 22 Nov, Fall 8 : 25 Nov to 6 Dec, Spring 9 : 25 May to 8 Jun, Summer 3: 6 Jul - 17 Jul, Summer 4: 14 Jul - 28 Jul, Summer 6: 17 Aug - 28 Aug, Summer 7: 31 Aug - 11 Sep, Fall 3: Oct 12 - Oct 23, Winter 5 Feb 22 - Mar 5, Winter 6 Mar 8 - Mar 19, Spring 4 2021 May 3 - May 14, Spring 5 2021 May 17 - May 28, Summer 2 2023 May 29, Fall 4 2023 Oct 16, Spring 2, Spring 5, Fall 1, Fall 5, Fall 6, Fall 7, Fall 8 [ 55, 56, 58, 59, 60, 66, 67, 68, 76, 77, 78, 79, 95, 98, 99, 101, 102, 106, 115, 116, 120, 121, 171, 180, 186, 189, 202, 206, 207, 208, 209 ]
                aloraine Ann Loraine made changes -
                Rank Ranked higher
                pbhatia1 Pranav Bhatia (Inactive) made changes -
                Status To-Do [ 10305 ] In Progress [ 3 ]
                pbhatia1 Pranav Bhatia (Inactive) made changes -
                Attachment screenshot-1.png [ 18601 ]
                Hide
                Permalink
                pbhatia1 Pranav Bhatia (Inactive) added a comment -

                Added my name is the contributors list.
                Successfully triggered the pipeline and downloaded the .dmg file to verify my changes.
                Attaching screenshot below for your reference -

                *Commit - * https://bitbucket.org/pranavbhatia1999/integrated-genome-browser/commits/fe8ee42f35b1ea69b95b9ac69e0745aa9a0880ad

                *Downloads - * https://bitbucket.org/pranavbhatia1999/integrated-genome-browser/downloads/

                Show
                pbhatia1 Pranav Bhatia (Inactive) added a comment - Added my name is the contributors list. Successfully triggered the pipeline and downloaded the .dmg file to verify my changes. Attaching screenshot below for your reference - *Commit - * https://bitbucket.org/pranavbhatia1999/integrated-genome-browser/commits/fe8ee42f35b1ea69b95b9ac69e0745aa9a0880ad *Downloads - * https://bitbucket.org/pranavbhatia1999/integrated-genome-browser/downloads/
                pbhatia1 Pranav Bhatia (Inactive) made changes -
                Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
                pbhatia1 Pranav Bhatia (Inactive) made changes -
                Assignee Pranav Bhatia [ pbhatia1 ]
                pkulzer Paige Kulzer (Inactive) made changes -
                Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
                pkulzer Paige Kulzer (Inactive) made changes -
                Assignee Paige Kulzer [ pkulzer ]
                Hide
                Permalink
                pkulzer Paige Kulzer (Inactive) added a comment -

                The commit message contains the ticket number and uses imperative voice, and only one line of code has been changed to add Pranav's name to the credits. Ready for PR!

                Show
                pkulzer Paige Kulzer (Inactive) added a comment - The commit message contains the ticket number and uses imperative voice, and only one line of code has been changed to add Pranav's name to the credits. Ready for PR!
                pkulzer Paige Kulzer (Inactive) made changes -
                Status First Level Review in Progress [ 10301 ] Ready for Pull Request [ 10304 ]
                pkulzer Paige Kulzer (Inactive) made changes -
                Assignee Paige Kulzer [ pkulzer ] Pranav Bhatia [ pbhatia1 ]
                pbhatia1 Pranav Bhatia (Inactive) made changes -
                Status Ready for Pull Request [ 10304 ] Pull Request Submitted [ 10101 ]
                pbhatia1 Pranav Bhatia (Inactive) made changes -
                Assignee Pranav Bhatia [ pbhatia1 ]
                Hide
                Permalink
                pbhatia1 Pranav Bhatia (Inactive) added a comment -

                Thanks Paige.
                PR created.

                Show
                pbhatia1 Pranav Bhatia (Inactive) added a comment - Thanks Paige. PR created.
                aloraine Ann Loraine made changes -
                Status Pull Request Submitted [ 10101 ] Reviewing Pull Request [ 10303 ]
                aloraine Ann Loraine made changes -
                Assignee Ann Loraine [ aloraine ]
                Hide
                Permalink
                aloraine Ann Loraine added a comment -

                Oh no!

                Because I merged an edit by saideepthi jagarapu affecting the same line in the same file, your edit now has a conflict. Unfortunately this means you must re-apply your edit to the new tip of the main branch and update your PR accordingly.

                Also, for future reference, please provide a link to your PRs in future tickets, as a convenience for reviewers.

                Show
                aloraine Ann Loraine added a comment - Oh no! Because I merged an edit by saideepthi jagarapu affecting the same line in the same file, your edit now has a conflict. Unfortunately this means you must re-apply your edit to the new tip of the main branch and update your PR accordingly. Also, for future reference, please provide a link to your PRs in future tickets, as a convenience for reviewers.
                aloraine Ann Loraine made changes -
                Status Reviewing Pull Request [ 10303 ] To-Do [ 10305 ]
                aloraine Ann Loraine made changes -
                Assignee Ann Loraine [ aloraine ] Pranav Bhatia [ pbhatia1 ]
                pbhatia1 Pranav Bhatia (Inactive) made changes -
                Status To-Do [ 10305 ] In Progress [ 3 ]
                Hide
                Permalink
                pbhatia1 Pranav Bhatia (Inactive) added a comment -

                Noted. Dr. Loranine. Will follow the same for future tickets.

                Show
                pbhatia1 Pranav Bhatia (Inactive) added a comment - Noted. Dr. Loranine. Will follow the same for future tickets.
                pbhatia1 Pranav Bhatia (Inactive) made changes -
                Attachment screenshot-2.png [ 18605 ]
                Hide
                Permalink
                pbhatia1 Pranav Bhatia (Inactive) added a comment -

                Resolved the conflict and added my name.
                Successfully triggered the pipeline and downloaded the .dmg file to verify my changes.

                Attaching screenshot below for your reference -

                Commit - https://bitbucket.org/pranavbhatia1999/integrated-genome-browser/commits/79c6b03aa98737dc87383cbc7deec294f50d2862

                Downloads - https://bitbucket.org/pranavbhatia1999/integrated-genome-browser/downloads/

                Show
                pbhatia1 Pranav Bhatia (Inactive) added a comment - Resolved the conflict and added my name. Successfully triggered the pipeline and downloaded the .dmg file to verify my changes. Attaching screenshot below for your reference - Commit - https://bitbucket.org/pranavbhatia1999/integrated-genome-browser/commits/79c6b03aa98737dc87383cbc7deec294f50d2862 Downloads - https://bitbucket.org/pranavbhatia1999/integrated-genome-browser/downloads/
                pbhatia1 Pranav Bhatia (Inactive) made changes -
                Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
                pbhatia1 Pranav Bhatia (Inactive) made changes -
                Assignee Pranav Bhatia [ pbhatia1 ]
                Hide
                Permalink
                aloraine Ann Loraine added a comment - - edited

                Thank you for updating your branch!
                I see that because you updated your branch, your existing PR as shown in the Bitbucket interface is no longer shown as having a conflict.

                The link to the PR is: https://bitbucket.org/lorainelab/integrated-genome-browser/pull-requests/1058

                I am proceeding to merge the PR, and moving this ticket forward on the board accordingly.

                Show
                aloraine Ann Loraine added a comment - - edited Thank you for updating your branch! I see that because you updated your branch, your existing PR as shown in the Bitbucket interface is no longer shown as having a conflict. The link to the PR is: https://bitbucket.org/lorainelab/integrated-genome-browser/pull-requests/1058 I am proceeding to merge the PR, and moving this ticket forward on the board accordingly.
                aloraine Ann Loraine made changes -
                Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
                aloraine Ann Loraine made changes -
                Status First Level Review in Progress [ 10301 ] Ready for Pull Request [ 10304 ]
                aloraine Ann Loraine made changes -
                Status Ready for Pull Request [ 10304 ] Pull Request Submitted [ 10101 ]
                aloraine Ann Loraine made changes -
                Status Pull Request Submitted [ 10101 ] Reviewing Pull Request [ 10303 ]
                aloraine Ann Loraine made changes -
                Assignee Ann Loraine [ aloraine ]
                Hide
                Permalink
                aloraine Ann Loraine added a comment -

                PR is merged and new main branch installers are built and deployed to bioviz.org early access section.
                Ready for testing.

                Show
                aloraine Ann Loraine added a comment - PR is merged and new main branch installers are built and deployed to bioviz.org early access section. Ready for testing.
                aloraine Ann Loraine made changes -
                Status Reviewing Pull Request [ 10303 ] Merged Needs Testing [ 10002 ]
                aloraine Ann Loraine made changes -
                Assignee Ann Loraine [ aloraine ]
                pkulzer Paige Kulzer (Inactive) made changes -
                Assignee Paige Kulzer [ pkulzer ]
                pkulzer Paige Kulzer (Inactive) made changes -
                Status Merged Needs Testing [ 10002 ] Post-merge Testing In Progress [ 10003 ]
                Hide
                Permalink
                pkulzer Paige Kulzer (Inactive) added a comment -

                Tested on Mac by downloading the Early Release installer, I'm now seeing Saideepthi's name in the credits but I'm not seeing Pranav's name like I would expect. I'm also not seeing his name in the credits when I download the installer from Bitbucket.

                I cloned his branch and tested his changes locally and was able to see his name in the credits that way, so I'm wondering if something went wrong after the merge conflict? [~ann.loraine] or Nowlan Freese, any ideas on how to fix this issue?

                Show
                pkulzer Paige Kulzer (Inactive) added a comment - Tested on Mac by downloading the Early Release installer, I'm now seeing Saideepthi's name in the credits but I'm not seeing Pranav's name like I would expect. I'm also not seeing his name in the credits when I download the installer from Bitbucket. I cloned his branch and tested his changes locally and was able to see his name in the credits that way, so I'm wondering if something went wrong after the merge conflict? [~ann.loraine] or Nowlan Freese , any ideas on how to fix this issue?
                pkulzer Paige Kulzer (Inactive) made changes -
                Status Post-merge Testing In Progress [ 10003 ] To-Do [ 10305 ]
                pkulzer Paige Kulzer (Inactive) made changes -
                Assignee Paige Kulzer [ pkulzer ] Pranav Bhatia [ pbhatia1 ]
                nfreese Nowlan Freese made changes -
                Status To-Do [ 10305 ] Pull Request Submitted [ 10101 ]
                nfreese Nowlan Freese made changes -
                Status Pull Request Submitted [ 10101 ] Reviewing Pull Request [ 10303 ]
                nfreese Nowlan Freese made changes -
                Status Reviewing Pull Request [ 10303 ] Merged Needs Testing [ 10002 ]
                nfreese Nowlan Freese made changes -
                Status Merged Needs Testing [ 10002 ] Post-merge Testing In Progress [ 10003 ]
                Hide
                Permalink
                nfreese Nowlan Freese added a comment - - edited

                I installed IGB (IGB-macos-universal-main.dmg) from loraine lab IGB downloads (https://bitbucket.org/lorainelab/integrated-genome-browser/downloads/) and BioViz.org early access and was able to see Pranav's name in the credits. I can also see that the commit is part of the main branch and his name appears in the AboutIGB.html file.

                I think this ticket is safe to close.

                Show
                nfreese Nowlan Freese added a comment - - edited I installed IGB (IGB-macos-universal-main.dmg) from loraine lab IGB downloads ( https://bitbucket.org/lorainelab/integrated-genome-browser/downloads/ ) and BioViz.org early access and was able to see Pranav's name in the credits. I can also see that the commit is part of the main branch and his name appears in the AboutIGB.html file. I think this ticket is safe to close.
                nfreese Nowlan Freese made changes -
                Resolution Done [ 10000 ]
                Status Post-merge Testing In Progress [ 10003 ] Closed [ 6 ]

                  People

                  • Assignee:
                    pbhatia1 Pranav Bhatia (Inactive)
                    Reporter:
                    aloraine Ann Loraine
                  • Votes:
                    0 Vote for this issue
                    Watchers:
                    4 Start watching this issue

                    Dates

                    • Created:
                      Updated:
                      Resolved: