Skip to content

Conversation

@ehennestad
Copy link
Collaborator

Fixed string conversion for empty objects to return empty string Add extra step in ControlledTerm to ensure scalar string "" is treated as empty

Fixed string conversion for empty objects to return empty string
Add extra step in ControlledTerm to ensure scalar string "" is treated as empty
@ehennestad ehennestad changed the title Handle empty and scalar cases in string conversion Fix string conversion for empty metadata instance objects Oct 24, 2025
@ehennestad ehennestad enabled auto-merge (squash) October 24, 2025 08:27
@github-actions
Copy link
Contributor

Test Results (R2021b)

558 tests  ±0   558 ✅ ±0   1m 40s ⏱️ +9s
 16 suites ±0     0 💤 ±0 
  1 files   ±0     0 ❌ ±0 

Results for commit 1edd8d0. ± Comparison against base commit eca1a2e.

@github-actions
Copy link
Contributor

Test Results (R2024b)

558 tests  ±0   558 ✅ ±0   1m 30s ⏱️ -4s
 16 suites ±0     0 💤 ±0 
  1 files   ±0     0 ❌ ±0 

Results for commit 1edd8d0. ± Comparison against base commit eca1a2e.

@codecov
Copy link

codecov bot commented Oct 24, 2025

Codecov Report

❌ Patch coverage is 50.00000% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 75.58%. Comparing base (eca1a2e) to head (1edd8d0).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
...openminds/+internal/+mixin/CustomInstanceDisplay.m 50.00% 2 Missing ⚠️
...ode/internal/+openminds/+abstract/ControlledTerm.m 50.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #77      +/-   ##
==========================================
- Coverage   75.61%   75.58%   -0.03%     
==========================================
  Files         344      344              
  Lines        3691     3695       +4     
==========================================
+ Hits         2791     2793       +2     
- Misses        900      902       +2     

☔ View full report in Codecov by Sentry.
📢 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.

@ehennestad ehennestad merged commit 834e8ba into main Oct 24, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants