Skip to content
This repository was archived by the owner on May 3, 2026. It is now read-only.

Clarify update return type#60

Merged
brothercorvo merged 2 commits into
mainfrom
codex/update-docstring-for-update-method
Aug 11, 2025
Merged

Clarify update return type#60
brothercorvo merged 2 commits into
mainfrom
codex/update-docstring-for-update-method

Conversation

@brothercorvo

Copy link
Copy Markdown
Member

Summary

  • Document BaseModel.update returning an updated dataclass instance or None
  • Note dataclass return types in emergency controller update endpoints
  • Add tests asserting update yields dataclass objects

Testing

  • flake8 reticulum_openapi/model.py examples/EmergencyManagement/Server/controllers_emergency.py tests/test_persistence.py tests/test_model.py
  • pytest

https://chatgpt.com/codex/tasks/task_e_689526c6a7b0832591f3fe014ffb3b11

@brothercorvo brothercorvo merged commit c8ce518 into main Aug 11, 2025
0 of 2 checks passed
@brothercorvo brothercorvo deleted the codex/update-docstring-for-update-method branch November 14, 2025 18:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant