Skip to content

fix ruff failures#110

Merged
blkdmr merged 1 commit into
pyfenn:mainfrom
hendo-21:fix/ruff-failures
May 29, 2026
Merged

fix ruff failures#110
blkdmr merged 1 commit into
pyfenn:mainfrom
hendo-21:fix/ruff-failures

Conversation

@hendo-21
Copy link
Copy Markdown
Contributor

Approach:

  • Used pre-commit run --all-files to address all but 4 ruff check errors. pre-commit resolved ruff format failure.
  • Manually applied fixes for code style violations that Ruff flags as unsafe for auto-fixing in the following files:
    • src/fenn/agents/__init__.py: suppressed E402 error in-line with noqa since import placement appeared intentional to avoid circular import
    • src/fenn/agents/node.py: changed variable names so no longer ambiguous
    • src/fenn/agents/rag/retriever.py: suppressed F401 error in-line with noqa since numpy import seems required for cofone dependency check

Copy link
Copy Markdown
Collaborator

@blkdmr blkdmr left a comment

Choose a reason for hiding this comment

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

Thank you very much for your work!

@blkdmr blkdmr merged commit 3052b45 into pyfenn:main May 29, 2026
1 of 2 checks passed
@hendo-21 hendo-21 deleted the fix/ruff-failures branch May 29, 2026 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants