Skip to content

Minor fixes #436

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
mojodna wants to merge 3 commits into
base: dev
Choose a base branch
from
Open

Minor fixes #436

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
1eefa88
fix(core): switch to relative import
mojodna Jan 28, 2026
38e0060
fix(core): fix __name__ reference
mojodna Jan 28, 2026
aa9a6ca
chore: add install make target
mojodna Jan 28, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions Makefile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,8 @@

default: test-all

install: uv-sync

uv-sync:
@uv sync --all-packages 2> /dev/null

Expand Down
4 changes: 2 additions & 2 deletions packages/overture-schema-core/src/overture/schema/core/models.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,6 @@
from pydantic_core import core_schema
from typing_extensions import Self

from overture.schema.core.sources import Sources
from overture.schema.system.feature import Feature
from overture.schema.system.field_constraint import UniqueItemsConstraint
from overture.schema.system.model_constraint import no_extra_fields
Expand All @@ -25,6 +24,7 @@
)

from .enums import PerspectiveMode
from .sources import Sources
from .types import (
FeatureVersion,
Level,
Expand Down Expand Up @@ -66,7 +66,7 @@ def __validate_ext_fields__(self) -> Self:
raise ValueError(
f"invalid extra field name{maybe_plural}: {', '.join(invalid_extra_fields)} "
"(extra fields are temporarily allowed, but only if their names start with 'ext_', "
"but all extra field name support in {self.__class__.name} is on a deprecation path "
"but all extra field name support in {self.__class__.__name__} is on a deprecation path"
"and will be removed)"
)
return self
Expand Down