Fix import formatting in models.py#1247
Fix import formatting in models.py#1247RisingOrange merged 2 commits intoclaude/remove-aqt-import-v1YsXfrom
Conversation
…orts Ruff I001 was failing because the typing import ran directly into the mashumaro import without a blank line separator. https://claude.ai/code/session_01K8GeVjG7MjJzhZMtrzRh48
Replace `lambda mid: {}` (which would crash with KeyError if the mock
were removed) with a proper empty notetype dict `{"css": "", "tmpls": []}`.
This removes the implicit dependency on the get_media_names_from_notetype
mock and makes each call site self-sufficient.
https://claude.ai/code/session_01K8GeVjG7MjJzhZMtrzRh48
Summary of ChangesHello @RisingOrange, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request primarily focuses on improving code style and test maintainability. It addresses import formatting in a core models file to enhance readability and consistency, while also refactoring test helper logic in a client test file to make the test suite clearer and more robust. Highlights
🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console. Changelog
Activity
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
* Remove aqt import from client models.py Refactor get_media_names_from_notetype to accept a note type dict instead of an ID, removing the aqt dependency from the client layer. Add a get_notetype_by_id callback parameter to get_media_names_from_notes_data so callers in the GUI/business layer provide the lookup function. https://claude.ai/code/session_01ENEZrvFPchRVdnATauZfo9 * Apply suggestion from @RisingOrange * Fix import formatting in models.py (#1247) * Fix import sorting: add blank line between stdlib and third-party imports Ruff I001 was failing because the typing import ran directly into the mashumaro import without a blank line separator. https://claude.ai/code/session_01K8GeVjG7MjJzhZMtrzRh48 * Remove fragile get_media_names_from_notetype mock in client tests Replace `lambda mid: {}` (which would crash with KeyError if the mock were removed) with a proper empty notetype dict `{"css": "", "tmpls": []}`. This removes the implicit dependency on the get_media_names_from_notetype mock and makes each call site self-sufficient. https://claude.ai/code/session_01K8GeVjG7MjJzhZMtrzRh48 --------- Co-authored-by: Claude <noreply@anthropic.com> * fix mypy errors (#1248) * Fix import sorting: add blank line between stdlib and third-party imports Ruff I001 was failing because the typing import ran directly into the mashumaro import without a blank line separator. https://claude.ai/code/session_01K8GeVjG7MjJzhZMtrzRh48 * Remove fragile get_media_names_from_notetype mock in client tests Replace `lambda mid: {}` (which would crash with KeyError if the mock were removed) with a proper empty notetype dict `{"css": "", "tmpls": []}`. This removes the implicit dependency on the get_media_names_from_notetype mock and makes each call site self-sufficient. https://claude.ai/code/session_01K8GeVjG7MjJzhZMtrzRh48 * Fix mypy errors: wrap mid with NotetypeId() for ModelManager.get() calls https://claude.ai/code/session_01K8GeVjG7MjJzhZMtrzRh48 --------- Co-authored-by: Claude <noreply@anthropic.com> --------- Co-authored-by: Claude <noreply@anthropic.com>
2 participants
Related issues
Proposed changes
This PR fixes the import statement formatting in
ankihub/ankihub_client/models.pyby adding a blank line after the standard library imports and before the third-party imports, following PEP 8 style guidelines.The change improves code readability and consistency by properly separating import groups:
datetime,enum,typing)mashumaro)How to reproduce
No manual testing needed - this is a code style fix.
Further comments
This is a minor formatting improvement that aligns with PEP 8 conventions for organizing imports. The blank line separates standard library imports from third-party library imports, making the code more readable and maintainable.
https://claude.ai/code/session_01K8GeVjG7MjJzhZMtrzRh48