Skip to content

add missing types to Ash.load typespec#2167

Merged
zachdaniel merged 3 commits intomainfrom
fix/ash-load-spec
Jun 27, 2025
Merged

add missing types to Ash.load typespec#2167
zachdaniel merged 3 commits intomainfrom
fix/ash-load-spec

Conversation

@barnabasJ
Copy link
Contributor

Contributor checklist

Leave anything that you believe does not apply unchecked.

  • Bug fixes include regression tests
  • Chores
  • Documentation changes
  • Features include unit/acceptance tests
  • Refactoring
  • Update dependencies

@barnabasJ barnabasJ self-assigned this Jun 26, 2025
@barnabasJ barnabasJ requested a review from zachdaniel June 26, 2025 12:06
zachdaniel
zachdaniel previously approved these changes Jun 26, 2025
"""
@spec load(
record_or_records :: Ash.Resource.record() | [Ash.Resource.record()] | nil,
record_or_records ::
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, can we also do this for load!?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, my bad. I also reused some of the already defined types in the process.

@barnabasJ barnabasJ force-pushed the fix/ash-load-spec branch from 481f013 to b96cbca Compare June 27, 2025 08:49
@barnabasJ barnabasJ requested a review from zachdaniel June 27, 2025 10:14
@zachdaniel zachdaniel merged commit b4de72b into main Jun 27, 2025
43 checks passed
@zachdaniel
Copy link
Contributor

🚀 Thank you for your contribution! 🚀

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.

2 participants