Skip to content

docs: add info about deadlocks in tests to usage rules#2241

Merged
zachdaniel merged 4 commits intomainfrom
deadlock_usage_rules
Jul 31, 2025
Merged

docs: add info about deadlocks in tests to usage rules#2241
zachdaniel merged 4 commits intomainfrom
deadlock_usage_rules

Conversation

@barnabasJ
Copy link
Copy Markdown
Contributor

Contributor checklist

Leave anything that you believe does not apply unchecked.

  • I accept the AI Policy, or AI was not used in the creation of this PR.
  • Bug fixes include regression tests
  • Chores
  • Documentation changes
  • Features include unit/acceptance tests
  • Refactoring
  • Update dependencies

…utes

Add new section to usage-rules.md explaining how to use Ash.Generator
to prevent postgres deadlock issues when running concurrent tests.
Includes examples for using sequence/3, creating reusable test
generators, and choosing between process-scoped and globally unique values.
Add usage_rules v0.1 to project dependencies for enhanced
development workflow support.
@barnabasJ barnabasJ self-assigned this Jul 31, 2025
@barnabasJ barnabasJ added the documentation Improvements or additions to documentation label Jul 31, 2025
@barnabasJ barnabasJ requested a review from zachdaniel July 31, 2025 10:26
mix.exs Outdated
{:benchee, "~> 1.1", only: [:dev, :test]},
{:tz, "~> 0.28", only: [:test]}
{:tz, "~> 0.28", only: [:test]},
{:usage_rules, "~> 0.1"}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This shouldn't be added here.

Copy link
Copy Markdown
Contributor

@zachdaniel zachdaniel left a comment

Choose a reason for hiding this comment

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

Can't add usage rules as a dependency like that (it would make igniter a prod dependency of downstream apps. If we added it it would need to be a dev dependency, but we don't need it 😄

@zachdaniel zachdaniel merged commit 1174f2a into main Jul 31, 2025
43 checks passed
@zachdaniel
Copy link
Copy Markdown
Contributor

🚀 Thank you for your contribution! 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants