Fix remaining ty type errors in baker.py and generators.py

[benaduo]

Summary

Fixes two remaining ty type checker errors not addressed by PR #584.

Changes

model_bakery/baker.py

• Added explicit list[type] annotation to other_fields_to_skip to fix invalid-argument-type when .extend([GenericRelation, GenericForeignKey]) is called
• Wrapped the .extend() call with cast(list[type], [...]) so ty is satisfied with the conditionally-imported types

model_bakery/generators.py

• Added explicit dict[type, Callable] annotation to default_mapping, replacing the trailing # type: dict[type, Callable] comment that ty does not recognise

Motivation

The CI linter job linters (ty check model_bakery) still fails on these two issues after #584. This PR resolves them completely.

Summary by CodeRabbit

• Refactor
  • Improved and standardized type annotations across core modules to enhance code clarity and type safety without changing behavior.
• Chores
  • Minor formatting and declaration cleanups to align code style and annotations.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 7ad80b95-fda3-40b6-b8d0-6e78875e3e66

📥 Commits

Reviewing files that changed from the base of the PR and between 1a178a9 and 6a3932d.

📒 Files selected for processing (2)

• model_bakery/baker.py
• model_bakery/generators.py

✅ Files skipped from review due to trivial changes (2)

• model_bakery/baker.py
• model_bakery/generators.py

---

📝 Walkthrough

Walkthrough

Small, non-functional edits adding explicit type annotations: other_fields_to_skip declared as list[type] in model_bakery/baker.py (uses cast(list[type], ...) when extending with contenttypes types), and default_mapping annotated as dict[type, Callable] in model_bakery/generators.py. No logic changes.

Changes

Cohort / File(s)	Summary
Type annotation updates model_bakery/baker.py, model_bakery/generators.py	Added explicit annotations: other_fields_to_skip: list[type] in baker.py and default_mapping: dict[type, Callable] in generators.py. Wrapped contenttypes list with cast(list[type], ...) before extend. No behavioral changes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

• PR #572: Similar typing/annotation adjustments in baker.py and related modules.

Poem

🐇 I hopped through code, made types clear and neat,
Casts tucked in places where generics greet,
Mappings labeled, no more hidden lore,
Tiny edits, tidy core,
A rabbit's whisper: "types, complete!"

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ 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%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ 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 accurately summarizes the main change: fixing remaining type checker (ty) errors in two specific files (baker.py and generators.py).

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

📝 Coding Plan

• Generate coding plan for human review comments

---

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.}

[amureki]

Thanks for picking up the fixes here! 🙏

[amureki]

Honestly, a bit weird to cast here, I'd be also fine with just excluding it # type: ignore[list-item], but okay.