Skip to content

raise error if the constraint is created on a field that doesn't exist#579

Open
xantrac wants to merge 1 commit intoash-project:mainfrom
xantrac:xantrac-error-constraint
Open

raise error if the constraint is created on a field that doesn't exist#579
xantrac wants to merge 1 commit intoash-project:mainfrom
xantrac:xantrac-error-constraint

Conversation

@xantrac
Copy link

@xantrac xantrac commented Jun 30, 2025

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
@zachdaniel
Copy link
Contributor

Thanks for the PR! I think this behavior is valuable but would be better placed in a verifier on the data layer extension. This will give errors at compile time instead of when generating migrations. See this verifier for example: https://github.com/ash-project/ash_postgres/blob/main/lib/verifiers/validate_identity_index_names.ex

@xantrac
Copy link
Author

xantrac commented Jun 30, 2025

Thanks for the PR! I think this behavior is valuable but would be better placed in a verifier on the data layer extension. This will give errors at compile time instead of when generating migrations. See this verifier for example: https://github.com/ash-project/ash_postgres/blob/main/lib/verifiers/validate_identity_index_names.ex

Sounds good!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants