Skip to content

Conversation

@totaltrash
Copy link
Contributor

@totaltrash totaltrash commented Jan 7, 2021

Contributor checklist

  • Bug fixes include regression tests
  • Features include unit/acceptance tests

Before merging this, I'd like to change the default behaviour of integer_primary_key over in ash-project/ash. It currently defaults allow_nil? to true, which is incompatible with :serial

fixes #32

@zachdaniel
Copy link
Contributor

Made that change you mentioned in integer_primary_key, going to go ahead and merge this.

🚀 Thank you for your contribution! 🚀

@zachdaniel zachdaniel closed this Jan 8, 2021
@zachdaniel zachdaniel reopened this Jan 8, 2021
@zachdaniel zachdaniel merged commit 650896a into ash-project:master Jan 8, 2021
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.

Migration generator doesn't consider generated?-flag on integer values

2 participants