Skip to content

🌍 Lots of simplifications to the Property model#327

Merged
nth-commit merged 5 commits intomainfrom
terra/property-simplifications
Jul 15, 2022
Merged

🌍 Lots of simplifications to the Property model#327
nth-commit merged 5 commits intomainfrom
terra/property-simplifications

Conversation

@nth-commit
Copy link
Copy Markdown
Owner

  • 🌍 Make Property.Reflect return Property, de-nest TestOutput/TestResult
  • 🌍 Expose ReadOnlyList for property input instead of Array
  • 🌍 Make Property inherit from Property, instead of the other way around
  • 🌍 Simplify lots of things by making Test implement Test<object?>
  • 🌍 Remove support for nested properties for now (simplification), may come back in a different form in the future

@nth-commit nth-commit force-pushed the terra/property-simplifications branch from 0d0fc7c to a3dd93e Compare July 15, 2022 21:52
@nth-commit nth-commit force-pushed the terra/property-simplifications branch from a3dd93e to fec2aa0 Compare July 15, 2022 21:59
@nth-commit nth-commit merged commit 8673dcc into main Jul 15, 2022
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.

1 participant