tracehubpm / code-review-action

Quality of Code Review Checker, plugin it as GitHub Action
MIT License
7 stars 2 forks source link

feat(#23): AnalysisPrompt, SystemPrompt #59

Closed h1alexbel closed 7 months ago

h1alexbel commented 7 months ago

closes #23 History:


PR-Codex overview

This PR introduces a new maxClassNameLen configuration in the pom.xml, adds analysis prompts for code reviews, and defines a system prompt for quality analysis.

Detailed summary

The following files were skipped due to too many changes: src/test/java/git/tracehub/codereview/action/prompt/AnalysisPromptTest.java

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

h1alexbel commented 7 months ago

@rultor merge

codecov[bot] commented 7 months ago

Codecov Report

Attention: Patch coverage is 76.66667% with 7 lines in your changes are missing coverage. Please review.

Project coverage is 52.66%. Comparing base (c32b5e8) to head (e77a0c9).

Files Patch % Lines
...it/tracehub/codereview/action/AnalysisRoutine.java 0.00% 7 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #59 +/- ## ============================================ + Coverage 47.14% 52.66% +5.51% - Complexity 22 27 +5 ============================================ Files 14 16 +2 Lines 140 169 +29 Branches 5 5 ============================================ + Hits 66 89 +23 - Misses 71 77 +6 Partials 3 3 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

rultor commented 7 months ago

@rultor merge

@h1alexbel OK, I'll try to merge now. You can check the progress of the merge here

rultor commented 7 months ago

@rultor merge

@h1alexbel Done! FYI, the full log is here (took me 3min)

h1alexbel commented 7 months ago

@rultor release, tag is 0.0.18

rultor commented 7 months ago

@rultor release, tag is 0.0.18

@h1alexbel OK, I will release it now. Please check the progress here

rultor commented 7 months ago

@rultor release, tag is 0.0.18

@h1alexbel Done! FYI, the full log is here (took me 3min)