Add app.config requirement to Mix tasks

[renatomassaro]

This supports applications defining contexts in the config/runtime.exs file.

Summary by CodeRabbit

• Bug Fixes
  • Ensured database schema dump and migration tasks automatically load application configuration before execution, preventing failures due to missing settings.
  • Improved reliability of running these tasks across environments by enforcing the required configuration load step.

[coderabbitai]

Walkthrough

Adds @requirements ["app.config"] to two Mix tasks (FeebDb.DumpSchemas and FeebDb.Migrate) so app.config is loaded before task execution. No other logic or control-flow changes.

Changes

Cohort / File(s)	Summary
Mix tasks: require app.config lib/mix/task/feeb_db/dump_schemas.ex, lib/mix/task/feeb_db/migrate.ex	Added module attribute @requirements ["app.config"] to ensure app.config loads before task runs; no other behavior or signatures changed.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

I twitch my whiskers, configs aligned,
Two tasks now wait till app’s assigned.
A hop, a bop, no code rearranged—
Just keys in place, dependencies ranged.
Carrot in paw, I nod with cheer—
Requirements set; all clear, my dear. 🥕✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	The title succinctly describes the key change of adding an app.config requirement to Mix tasks, which aligns directly with the modifications to the DumpSchemas and Migrate modules. It is concise, specific to the main change, and avoids extraneous information.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch add-appconfig-requirement-to-mix-tasks

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e96ba64 and 446e10a.

⛔ Files ignored due to path filters (2)

• lib/mix/task/feeb_db/gen/migration.ex is excluded by !**/gen/**
• lib/mix/task/feeb_db/gen/query.ex is excluded by !**/gen/**

📒 Files selected for processing (2)

• lib/mix/task/feeb_db/dump_schemas.ex (1 hunks)
• lib/mix/task/feeb_db/migrate.ex (1 hunks)

🔇 Additional comments (2)

lib/mix/task/feeb_db/dump_schemas.ex (1)

5-6: Approve code changes

The @requirements ["app.config"] attribute is correctly placed and ensures the application’s runtime configuration (e.g. contexts from config/runtime.exs) is loaded before task execution. Since the mix CLI isn’t available in this review environment, please manually verify that:

mix feeb_db.dump_schemas

runs successfully without configuration errors.

lib/mix/task/feeb_db/migrate.ex (1)

6-7: Approve mix task @requirements; manual runtime config check

The @requirements ["app.config"] attribute ensures runtime configuration is loaded before calling Boot.migrate_shards(). This aligns with lib/mix/task/feeb_db/dump_schemas.ex. Please manually verify that mix feeb_db.migrate succeeds under your config/runtime.exs without configuration errors.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}