Skip to content

FIX: Windows portability for CI collection - #68

Merged
RBhupi merged 7 commits into
mainfrom
devel
Jul 28, 2026
Merged

FIX: Windows portability for CI collection#68
RBhupi merged 7 commits into
mainfrom
devel

Conversation

@RBhupi

@RBhupi RBhupi commented Jul 27, 2026

Copy link
Copy Markdown
Collaborator

PR tags (required)

Select all that apply.

  • docs-only
  • tests-only
  • tiny-change (<5 lines, single class/file)
  • code-change
  • scientific/algorithmic
  • design/architecture
  • GUI

What changed

  • Windows issues are resolved for CICD

Validation

Mark each item as done or N/A. Add one short evidence line for each item.

  • I ran ruff

    • Evidence (command + pass/fail):
    • N/A reason (if not run):

  • I ran import-linter

    • Evidence (command + pass/fail):
    • N/A reason (if not run):

  • I ran tests

    • Evidence (command + pass/fail):
    • N/A reason (if not run):

AI usage & manual verification

If AI tools were used, complete this section.

  • AI used: yes
  • Model/tooling: Cluade Opus 4.8
  • Tasks delegated to AI: Coding
  • Manual verification performed: yes.

Reviewer guide

This should pass the CICD pipeline later we will test it on real windows machine.

@RBhupi RBhupi changed the title Devel FIX: Windows portability for CI collection Jul 27, 2026
@codecov-commenter

codecov-commenter commented Jul 27, 2026 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 78.66667% with 16 lines in your changes missing coverage. Please review.
✅ Project coverage is 87.21%. Comparing base (3024f5d) to head (61fc9ae).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
src/adapt/runtime/orchestrator.py 25.00% 9 Missing ⚠️
src/adapt/api/client.py 20.00% 4 Missing ⚠️
src/adapt/utils/process.py 92.85% 2 Missing ⚠️
src/adapt/persistence/execution_history.py 0.00% 1 Missing ⚠️
Additional details and impacted files 
@@            Coverage Diff             @@
##             main      #68      +/-   ##
==========================================
+ Coverage   87.19%   87.21%   +0.01%     
==========================================
  Files         110      111       +1     
  Lines        6953     6970      +17     
  Branches      899      903       +4     
==========================================
+ Hits         6063     6079      +16     
- Misses        611      612       +1     
  Partials      279      279
Flag Coverage Δ
unittests 87.20% <77.33%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

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

@RBhupi
RBhupi merged commit a53e00f into main Jul 28, 2026
9 checks passed
This was referenced Jul 28, 2026
Closed
Merged
RBhupi added a commit that referenced this pull request Jul 31, 2026
@RBhupi
DOC: Added conda installation; cleaned (#69)
3a30db2 
## PR tags (required)
- [x] docs-only
- [x] tiny-change (<5 lines, single class/file)

## What changed
`conda` installation command added.

## Issues closed
Closes #68 #9
@RBhupi RBhupi mentioned this pull request Jul 31, 2026
Closed
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@RBhupi @codecov-commenter