Changes between Version 29 and Version 30 of CodeReviewerPlugin
- Timestamp:
- Dec 24, 2016, 3:40:56 AM (7 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
CodeReviewerPlugin
v29 v30 46 46 1. Install the plugin, after downloading and unzipping: 47 47 {{{#!sh 48 cd codereviewerplugin/0.1249 sudo python setup.py bdist_egg50 sudo cp dist/TracCodeReviewer*.egg /your/trac/location/plugins/48 $ cd codereviewerplugin/0.12 49 $ sudo python setup.py bdist_egg 50 $ sudo cp dist/TracCodeReviewer*.egg /your/trac/location/plugins/ 51 51 }}} 52 52 … … 71 71 4. After the above, upgrade your Trac environment: 72 72 {{{#!sh 73 sudo trac-admin /path/to/projenv upgrade 73 $ sudo trac-admin /path/to/projenv upgrade 74 }}} 75 76 5. Resync the repository 77 {{{#!sh 78 $ sudo trac-admin /path/to/projenv codreviewer resync <repos> 74 79 }}} 75 80 … … 114 119 In the example above, each time a code review is submitted for a completed ticket, the {{{stage_deploy}}} Jenkins job gets triggered. The job would then call [browser:codereviewerplugin/0.12/coderev/util/reviewer.py Reviewer.get_next_changeset()] to determine what git changeset to deploy (if any). Note that this trigger for, say, a stage environment would be in addition to any trigger for running tests against the HEAD of your branch, or however you may currently have your CI system set up. 115 120 116 ==== Ticket-changeset map117 118 In order for the [browser:codereviewerplugin/0.12/coderev/util/reviewer.py Reviewer] to know all changesets that reference a given ticket, it needs a reliable ticket-changeset map. The built-in {{{revision}}} table is unfortunately not sufficiently reliable for some version control systems, eg Git. This is a known issue. So this plugin comes with its own changeset listener that manages a new {{{codereviewer_map}}} table. For existing changesets, you can run a provided [browser:codereviewerplugin/0.12/coderev/util/sync.py sync.py] utility once to populate the table, but should not need it after that.119 120 121 ==== Completeness criteria 121 122