Skip to content

fixing prop name canonicalization - #402

Merged
keighrim merged 2 commits into
developfrom
401-fix-alias-canonicalization
Oct 4, 2026
Merged

keighrim merged 2 commits into
developfrom
401-fix-alias-canonicalization

Conversation

@keighrim

@keighrim keighrim commented Oct 4, 2026 •

Copy link
Copy Markdown
Member

fixes #401

@clams-bot clams-bot added this to infra Oct 4, 2026
@github-project-automation github-project-automation Bot moved this to Todo in infra Oct 4, 2026
@keighrim
keighrim force-pushed the 401-fix-alias-canonicalization branch from 5cb0369 to 0736e1e Compare October 4, 2026 21:14
@codecov

codecov Bot commented Oct 4, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 87.50000% with 13 lines in your changes missing coverage. Please review.
✅ Project coverage is 77.34%. Comparing base (ecc064d) to head (0736e1e).

Files with missing lines Patch % Lines
mmif/utils/summarizer/nodes.py 72.22% 10 Missing ⚠️
mmif/utils/summarizer/summary.py 33.33% 2 Missing ⚠️
mmif/serialize/annotation.py 98.11% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #402      +/-   ##
===========================================
+ Coverage    75.04%   77.34%   +2.30%     
===========================================
  Files           27       27              
  Lines         3274     3311      +37     
===========================================
+ Hits          2457     2561     +104     
+ Misses         817      750      -67     
Flag Coverage Δ
unittests 77.34% <87.50%> (+2.30%) ⬆️

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.

@keighrim
keighrim merged commit 7dd9a3e into develop Oct 4, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

canonicalize aliased property names instead of mirroring them

2 participants