Skip to content

Restructuring#8

Open
sr-murthy wants to merge 31 commits into
mainfrom
code-reorganisation
Open

Restructuring#8
sr-murthy wants to merge 31 commits into
mainfrom
code-reorganisation

Conversation

@sr-murthy
Copy link
Copy Markdown
Contributor

@sr-murthy sr-murthy commented Apr 17, 2026

  • Tidy up pre-commit and test workflows ✅
  • An additional build workflow (build.yml) for building package artifacts (sdist, wheel) and a simple test of installing the package from the locally built wheel and testing its import with Python ✅
  • Use sensible runner timeouts for all jobs ✅
  • Upgrade all actions and pin them to full commit SHAs ✅
  • Tidy up project metadata in project TOML ✅
  • Update README ⏳
  • Tidy up and restructure core libs ✅
  • More unit tests + coverage reporting improvements ✅
  • Update Sphinx docs conf and add API reference to contents - RTD site (via PR build) accessible here

@sr-murthy sr-murthy self-assigned this Apr 17, 2026
@sr-murthy sr-murthy added ci Continuous integration packaging Packaging including management of project metadata and dependencies in the TOML. security Security labels Apr 17, 2026
Comment thread .github/workflows/build.yml Fixed
Comment thread .github/workflows/test.yml Fixed
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 17, 2026

Codecov Report

❌ Patch coverage is 37.97468% with 196 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
isaricanalytics/analytics.py 27.04% 116 Missing ⚠️
isaricanalytics/redcap_data.py 35.71% 72 Missing ⚠️
isaricanalytics/visualisation.py 75.00% 7 Missing ⚠️
isaricanalytics/utils.py 93.75% 1 Missing ⚠️
Files with missing lines Coverage Δ
isaricanalytics/__init__.py 100.00% <100.00%> (ø)
isaricanalytics/utils.py 93.75% <93.75%> (ø)
isaricanalytics/visualisation.py 60.10% <75.00%> (ø)
isaricanalytics/redcap_data.py 11.72% <35.71%> (ø)
isaricanalytics/analytics.py 11.33% <27.04%> (ø)

... and 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@sr-murthy sr-murthy force-pushed the code-reorganisation branch from 4d6087c to dd17cf8 Compare April 17, 2026 09:50
Comment thread .github/workflows/build.yml Fixed
Comment thread .github/workflows/test.yml Fixed
@sr-murthy sr-murthy force-pushed the code-reorganisation branch 4 times, most recently from 4129a84 to ac6799c Compare April 17, 2026 10:39
@sr-murthy sr-murthy changed the title Code reorganisation Restructuring Apr 17, 2026
@sr-murthy sr-murthy changed the title Restructuring DRAFT: Restructuring Apr 17, 2026
@sr-murthy sr-murthy marked this pull request as draft April 17, 2026 11:13
@sr-murthy sr-murthy added refactor Refactoring changes labels Apr 17, 2026
Comment thread .github/workflows/build.yml Fixed
Comment thread .github/workflows/test.yml Fixed
@sr-murthy sr-murthy force-pushed the code-reorganisation branch from 977ceb7 to 32369c2 Compare April 17, 2026 12:26
@sr-murthy sr-murthy linked an issue Apr 17, 2026 that may be closed by this pull request
@sr-murthy sr-murthy added the enhancement New feature or request label Apr 17, 2026
@sr-murthy sr-murthy force-pushed the code-reorganisation branch 5 times, most recently from 1193962 to 8b96d91 Compare April 21, 2026 11:00
sr-murthy added 30 commits May 11, 2026 15:24
…simple import test of the wheel installation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci Continuous integration documentation Improvements or additions to documentation enhancement New feature or request packaging Packaging including management of project metadata and dependencies in the TOML. refactor Refactoring changes security Security tests All test-related changes

Projects

None yet

3 participants