Skip to content

Conversation

@kybishop
Copy link
Contributor

@kybishop kybishop commented Dec 5, 2025

Screenshot 2025-12-05 at 1 17 44 PM

@kybishop kybishop requested review from joshk and nshoes December 5, 2025 18:19
@kybishop kybishop marked this pull request as draft December 5, 2025 20:41
@kybishop kybishop marked this pull request as ready for review December 6, 2025 19:15
@@ -0,0 +1,23 @@
defmodule NervesHub.Repo.Migrations.AddCascadeDeleteToDeploymentReleases do
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Foreign key validation does not need to be separated out to a second migration because there are no deployment_release records until after this PR.

Copy link
Contributor

@nshoes nshoes left a comment

Choose a reason for hiding this comment

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

This LGTM. I'll publish and run through a few tests in our QA.

@nshoes
Copy link
Contributor

nshoes commented Dec 9, 2025

Looks good!
CleanShot 2025-12-08 at 19 33 48@2x

@kybishop kybishop merged commit bc334d3 into main Dec 9, 2025
2 checks passed
@kybishop kybishop deleted the release-history branch December 9, 2025 17:35
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.

3 participants