New URL for NEMO forge!   http://forge.nemo-ocean.eu

Since March 2022 along with NEMO 4.2 release, the code development moved to a self-hosted GitLab.
This present forge is now archived and remained online for history.
Ticket Comment Diff – NEMO

Changes between Initial Version and Version 1 of Ticket #1948, comment 4


Ignore:
Timestamp:
2017-12-11T17:00:46+01:00 (6 years ago)
Author:
nicolasmartin
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #1948, comment 4

    initial v1  
    33Over time, the development workflow would become something close to the following: 
    44 
    5 1. The developer will push his changes to the repository with the new syntax for the commit message, `[see #1948]` as example. 
     51. The developer will push his changes to the repository with the new syntax for the commit message, `see #1948` as example. 
    662. Thanks to this, the commit will create a new comment in the ticket with the commit message and display the status of the review as 'Pending' for default. 
    773. The developer contacts the reviewer(s), the assignment of the ticket could be a clever method to do this. 
    884. The reviewer(s) put his review directly in the changeset page of the commit (see bottom of page), there is a textarea for comments and dropdown menu for status ('Pending' | 'Failed' | 'Passed') 
    9 5. The review submitted will become a new comment in the ticket and the `code_review` custom field will be updated appropriately regarding the review status. 
     95. The review submitted will become a new comment in the ticket and the `branch_review` custom field will be updated appropriately regarding the review status. 
    1010 
    11 The review can be modified and the developer can link as much commits as he wants to a development ticket but by design there will be only one status for `code_review` custom field. 
     11The review can be modified and the developer can link as much commits as he wants to a development ticket but by design there will be only one status for `branch_review` custom field.