Created demo modules for testing purposes. Jacoco report switched to …#86
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.
…overage reporting
…for pull requests
benedeki
left a comment
There was a problem hiding this comment.
coverage-per-changed-file: 60.0 might be steep for some files, but whatever.
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