Add codequality reports backend processing
Ref: #271077 (closed)
This feature will be behind a feature flag called codequality_mr_diff
What does this MR do?
This MR makes it possible to process a codequality
report on the backend and surface it at the pipeline level
It includes the following items:
-
Add new CodeClimate
Parser -
Introduce CodequalityReports
-
Surface codequality
report at the build level -
Surface codequality
report at the pipeline level
Why are we doing this?
Step #1
to introduce Codequality
Report comparison on diff/MR.
The grouptesting is currently moving the ability to generate diff report between the base and head pipeline of a codequality report to the backend.
Right now we are generating an inline diff on the frontend. We are planning to build up more features with codequality and one of those would be to display codequality diff on the MR itself. This will be similar to our current test coverage feature.
This implementation is similar to other report backend processing:
Screenshots
Context
This MR represents a vertical slice of functionality for moving our codequality
comparison feature on the backend . This is slice 1/3.
This work will occur in 3 slices:
- Move codequality processing on the backend - !47933 (merged)
- Introduce comparison logic -
- Surface diff report at the merge request level -
Does this MR meet the acceptance criteria?
Conformity
- [-] Changelog entry
- [-] Documentation (if required)
-
Code review guidelines - [-] Merge request performance guidelines
-
Style guides - [-] Database guides
- [-] Separation of EE specific content
Availability and Testing
- [-] Review and add/update tests for this feature/bug. Consider all test levels. See the Test Planning Process.
- [-] Tested in all supported browsers
- [-] Informed Infrastructure department of a default or new setting change, if applicable per definition of done
Security
If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:
- [-] Label as security and @ mention
@gitlab-com/gl-security/appsec
- [-] The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
- [-] Security reports checked/validated by a reviewer from the AppSec team