Skip to content

Fix warns, fix safetoinsert for only changing name - #1447

Merged
parsonsmatt merged 4 commits into
masterfrom
matt/fix-safe-to-insert
Dec 3, 2022
Merged

parsonsmatt merged 4 commits into
masterfrom
matt/fix-safe-to-insert

Conversation

@parsonsmatt

@parsonsmatt parsonsmatt commented Dec 3, 2022 •

Copy link
Copy Markdown
Collaborator

Fixes #1442

Before submitting your PR, check that you've:

  • Documented new APIs with Haddock markup
  • Added @since declarations to the Haddock
  • Ran stylish-haskell on any changed files.
  • Adhered to the code style (see the .editorconfig file for details)

After submitting your PR:

  • Update the Changelog.md file with a link to your PR
  • Bumped the version number if there isn't an (unreleased) on the Changelog
  • Check that CI passes (or if it fails, for reasons unrelated to your change, like CI timeouts)

@parsonsmatt
parsonsmatt merged commit 55d72a8 into master Dec 3, 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.

how to handle: PersistEntity ... does not have a default primary key? (persistent-2.14)

1 participant