Skip to content

Refactor HostSystem to improve maintainability#623

Merged
ascopes merged 1 commit into
mainfrom
task/refactor-host-system
Mar 31, 2025
Merged

Refactor HostSystem to improve maintainability#623
ascopes merged 1 commit into
mainfrom
task/refactor-host-system

Conversation

@ascopes

@ascopes ascopes commented Mar 31, 2025

Copy link
Copy Markdown
Owner
  • Scope per Mojo execution rather than per session for consistency with the rest of the project.
  • Use functional programming style for project property and env parsing.
  • Rescope test constructor.

- Scope per Mojo execution rather than per session
  for consistency with the rest of the project.
- Use functional programming style for project
  property and env parsing.
- Rescope test constructor.
@ascopes ascopes added the chore General tech debt work. label Mar 31, 2025
@ascopes ascopes added this to the v3.0.x milestone Mar 31, 2025
@ascopes ascopes self-assigned this Mar 31, 2025
@codecov

codecov Bot commented Mar 31, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.12%. Comparing base (e333aeb) to head (34dfbd9).
Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #623      +/-   ##
==========================================
+ Coverage   90.99%   91.12%   +0.14%     
==========================================
  Files          48       48              
  Lines        1497     1520      +23     
  Branches      100      100              
==========================================
+ Hits         1362     1385      +23     
  Misses         99       99              
  Partials       36       36              
Files with missing lines Coverage Δ
.../ascopes/protobufmavenplugin/utils/HostSystem.java 100.00% <100.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ascopes ascopes merged commit d842d4f into main Mar 31, 2025
@ascopes ascopes deleted the task/refactor-host-system branch March 31, 2025 06:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore General tech debt work.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant