Skip to content

fix(models): add dependent: options to has_many associations#391

Merged
natsuki615 merged 1 commit intomasterfrom
fix/lint-has-many-dependent
Mar 6, 2026
Merged

fix(models): add dependent: options to has_many associations#391
natsuki615 merged 1 commit intomasterfrom
fix/lint-has-many-dependent

Conversation

@merichar
Copy link
Copy Markdown
Member

@merichar merichar commented Mar 2, 2026

Fixes all 7 Rails/HasManyOrHasOneDependent violations by adding
explicit dependent: options to unguarded has_many associations

* OrganizationBuildStatus -> organization_build_steps: destroy
* Participant -> organization_build_steps (approver): restrict_with_error
* Participant -> events (assignee): nullify
* StoreItem -> store_purchases: restrict_with_error
* ToolInventory -> tool_inventory_tools: destroy
* ToolType -> tools: restrict_with_error
* ToolType -> tool_inventory_tools: destroy
@natsuki615 natsuki615 merged commit 01fcd74 into master Mar 6, 2026
1 check failed
@merichar merichar deleted the fix/lint-has-many-dependent branch March 19, 2026 20:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants