In current version (even after several iterations previously to deal with wiring issues/conflicts, etc.), I need to do several review cycle manually (using various review tools, e.g. ql-review, pr-review etc.), and iterate until everything looks good.
Hence it worth including a review cycles in the ql-execute so that we reduce the manual effort
also we need to pay attention to the context of the review:
- may refer to all the spec
- may look at implementation itself
- may not introduce unecessary new code (this is a very likely defect)
In current version (even after several iterations previously to deal with wiring issues/conflicts, etc.), I need to do several review cycle manually (using various review tools, e.g. ql-review, pr-review etc.), and iterate until everything looks good.
Hence it worth including a review cycles in the ql-execute so that we reduce the manual effort
also we need to pay attention to the context of the review: