Created demo modules for testing purposes. Jacoco report switched to …#86
Created demo modules for testing purposes. Jacoco report switched to …#86miroslavpojer wants to merge 15 commits into
Conversation
…master. Disabled other wf as not needed.
WalkthroughThe JaCoCo workflow sets job-level GitHub permissions, raises ChangesJaCoCo Workflow Updates
Sequence Diagram(s)sequenceDiagram
participant BuildJob as Build Job
participant SetupPython as actions/setup-python@v4
participant JacocoReport as MoranaApps/jacoco-report
Build Job->>SetupPython: install Python 3.14
Build Job->>JacocoReport: invoke with pinned ref and new inputs (global-thresholds, report-thresholds-default, comment-level:'full', skip-unchanged:true)
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
…, and technical debt - Created `operative_task.yml` for general operational and management tasks. - Introduced `pointer.yml` to track external work across repositories. - Added `spike_task.yml` for research and investigation tasks. - Implemented `technical_debt.yml` to document and address technical debt items. docs: Add Copilot instructions and review rules - Created `copilot-instructions.md` detailing coding guidelines, output discipline, and testing practices. - Added `copilot-review-rules.md` to define review behavior and commenting rules. chore: Configure Dependabot and GitHub workflows - Added `dependabot.yml` for automated dependency updates. - Created various workflows for issue assignment, build, release drafting, and publishing. - Implemented checks for release notes presence and JaCoCo code coverage. test: Add filename check workflow - Introduced `test_filenames_check.yml` to enforce naming conventions for test files.
| - name: Check coverage thresholds and add reports in PR comments | ||
| id: jacoco | ||
| uses: MoranaApps/jacoco-report@54bfe284d1119dc917dddba80517c54c5bcf3627 | ||
| uses: MoranaApps/jacoco-report@52a5f9c278aaf61700c9975e61a0853875c9d156 |
There was a problem hiding this comment.
Add in the comment, what version it is. E.g.
` uses: MoranaApps/jacoco-report@52a5f9c #v3.14
There was a problem hiding this comment.
I have addressed it, but when updated by dependabot the comment will be not changed.
…overage reporting
…for pull requests
Update JaCoCo report GitHub Action and add Python setup
Summary
This PR upgrades the
MoranaApps/jacoco-reportaction to its latest pinned commit and adds a required Python setup step to the JaCoCo CI workflow. No library source or test code is changed.Changes
.github/workflows/jacoco_report.ymlSetup Pythonstep (actions/setup-python@a309ff8..., Python 3.14) before the database setup — required by the newerjacoco-reportaction version.MoranaApps/jacoco-reportaction pin from54bfe28→52a5f9c.Release Notes
MoranaApps/jacoco-reportaction to latest stable commit (52a5f9c) for improved coverage reporting in PR comments.Summary by CodeRabbit
Closes #88