Skip to content

Fix: Remove unused pytest import#7

Merged
DecisionNerd merged 1 commit into
mainfrom
fix/remove-unused-pytest-import
Feb 1, 2026
Merged

Fix: Remove unused pytest import#7
DecisionNerd merged 1 commit into
mainfrom
fix/remove-unused-pytest-import

Conversation

@DecisionNerd
Copy link
Copy Markdown
Owner

@DecisionNerd DecisionNerd commented Feb 1, 2026

Summary

Remove unused pytest import from WITH clause integration tests that was causing lint failure.

Problem

Lint check failing on main with:

F401 [*] `pytest` imported but unused
  --> tests/integration/test_with_clause.py:10:8

Solution

Remove the unused import. The test file doesn't actually need pytest imported directly - it only needs GraphForge.

Testing

  • Lint check should pass
  • Tests still run correctly (pytest is invoked by test runner, not imported)

Checklist

  • Fix applied
  • No functional changes
  • Tests unaffected

Summary by CodeRabbit

  • Tests
    • Removed unused test dependencies.

✏️ Tip: You can customize this high-level summary in your review settings.

Fix lint error F401 - pytest was imported but never used.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@DecisionNerd DecisionNerd added tests release:none No release needed (docs/CI only) labels Feb 1, 2026
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Feb 1, 2026

Note

.coderabbit.yaml has unrecognized properties

CodeRabbit is using all valid settings from your configuration. Unrecognized properties (listed below) have been ignored and may indicate typos or deprecated fields that can be removed.

⚠️ Parsing warnings (1)
Validation error: Unrecognized key(s) in object: 'python', 'ignore', 'tools', 'file_types', 'comments', 'thresholds', 'integrations', 'notifications'
⚙️ Configuration instructions
  • Please see the configuration documentation for more information.
  • You can also validate your configuration using the online YAML validator.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Walkthrough

Removed an unused pytest import from a test file. No runtime logic or control flow changes. This is a cleanup modification that eliminates an unnecessary dependency declaration.

Changes

Cohort / File(s) Summary
Import cleanup
tests/integration/test_with_clause.py
Removed unused pytest import and adjacent blank line.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Description check ❓ Inconclusive The description provides summary, problem statement, and solution but lacks the structured template format with Type of Change, Related Issues, Changes Made, and Testing sections required by the repository template. Add the required template sections (Type of Change, Related Issues, Changes Made, Testing with test commands, and Checklist items) to match the repository's description format for consistency and completeness.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: removing an unused pytest import, which directly matches the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ 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 fix/remove-unused-pytest-import

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

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

@DecisionNerd DecisionNerd merged commit c562cad into main Feb 1, 2026
16 of 29 checks passed
@DecisionNerd DecisionNerd deleted the fix/remove-unused-pytest-import branch February 1, 2026 06:09
DecisionNerd added a commit that referenced this pull request May 4, 2026
- Create v0.3.10 release story (#448): NetworkX/igraph export + parse cache
- Move #391 to v0.3.10; reorder milestones (#6=v0.3.10, #7=v0.4.0)
- README roadmap table: mark v0.3.9 in-progress, add v0.3.10 row
- docs/releases/roadmap.md: full rewrite reflecting v0.3.8 released,
  v0.3.9 in-progress, v0.3.10/v0.4.0/v1.0 planned with accurate scope

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release:none No release needed (docs/CI only)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant