Skip to content

add cascade delete for github_app_installations and artifact_license_risks#2000

Merged
timbastin merged 2 commits into
mainfrom
feat/cascade-delete-github-app-installations-and-artifact-license-risks
May 20, 2026
Merged

add cascade delete for github_app_installations and artifact_license_risks#2000
timbastin merged 2 commits into
mainfrom
feat/cascade-delete-github-app-installations-and-artifact-license-risks

Conversation

@refoo0
Copy link
Copy Markdown
Member

@refoo0 refoo0 commented May 20, 2026

No description provided.

…cense_risks

Signed-off-by: rafi <refaei.shikho@hotmail.com>
Copilot AI review requested due to automatic review settings May 20, 2026 08:07
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates database foreign key constraints to cascade-delete dependent rows when their parent records are removed, reducing orphaned data in the DB.

Changes:

  • Add ON DELETE CASCADE to github_app_installations.org_id -> organizations.id.
  • Add ON DELETE CASCADE to artifact_license_risks -> license_risks foreign key.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Signed-off-by: rafi <refaei.shikho@hotmail.com>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@timbastin timbastin merged commit 5bc7ae5 into main May 20, 2026
15 checks passed
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