Closed jo-elimu closed 2 months ago
Attention: Patch coverage is 2.45902%
with 119 lines
in your changes missing coverage. Please review.
Project coverage is 15.15%. Comparing base (
835482e
) to head (97aed9b
). Report is 12 commits behind head on main.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@Souvik-Cyclic I added a note to the PR template that you created earlier.
The changes encompass the configuration of the spotless-maven-plugin
to enforce consistent code formatting across the project. The plugin is integrated into the Maven build process, enhancing the documentation for pull requests with guidance on running format checks. Additionally, minor formatting adjustments were made throughout various Java classes and test files, focusing on whitespace and indentation without altering functionality.
File(s) | Change Summary |
---|---|
.github/pull_request_template.md , .github/workflows/maven-build.yml |
Added documentation for format checks and a new CI job for code formatting. |
src/main/java/ai/elimu/dao/*.java , src/main/java/ai/elimu/util/*.java , src/main/java/ai/elimu/web/*.java , src/test/java/ai/elimu/dao/*.java , src/test/java/selenium/web/*.java |
Minor formatting changes, including whitespace adjustments and indentation corrections across various classes. |
Objective | Addressed | Explanation |
---|---|---|
Configure spotless-maven-plugin (#1728) |
âś… |
.github/workflows/maven-build.yml (1)
`33-48`: **Integration of `code_format_check` job is well-structured.** The addition of the `code_format_check` job is well-integrated into the existing workflow. It correctly depends on the `build` job and uses consistent setup steps for Java and Maven. The use of `spotless:check` ensures that code formatting is verified before the build process is finalized. This enhances the quality assurance process by maintaining code consistency.
closes #1728
pom.xml
https://github.com/elimu-ai/webapp/pull/1729/commits/65e9b09ad5c9bad3152be3d9db2391f01833bcfcmaven-build.yml
) https://github.com/elimu-ai/webapp/pull/1729/commits/41e3bcc0e5670f1c0df2d1482de0f14fcea1f9a7