Skip to content

fix: handle non-string binding labels (#38)#48

Open
cmungall wants to merge 1 commit into
mainfrom
fix/non-string-binding-label-38
Open

fix: handle non-string binding labels (#38)#48
cmungall wants to merge 1 commit into
mainfrom
fix/non-string-binding-label-38

Conversation

@cmungall

Copy link
Copy Markdown
Member

Summary

  • warn instead of crashing when a binding label field is not a string
  • support multivalued label fields by accepting any string label that matches the ontology label
  • keep mismatch warnings when no provided string label matches

Closes #38.

Tests

  • uv run pytest tests/test_plugins.py -q
  • just test

@claude

claude Bot commented Jun 18, 2026

Copy link
Copy Markdown

Claude finished @cmungall's task in 0s —— View job


I'll analyze this and get back to you.

@cmungall cmungall marked this pull request as ready for review June 18, 2026 20:50
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.

_validate_label can crash on a non-string label value

1 participant