Skip to content

Remove redundant dependencies#111

Open
rjavadi wants to merge 1 commit into
mainfrom
fix/pip-audit-deps
Open

Remove redundant dependencies#111
rjavadi wants to merge 1 commit into
mainfrom
fix/pip-audit-deps

Conversation

@rjavadi
Copy link
Copy Markdown
Collaborator

@rjavadi rjavadi commented May 21, 2026

Summary

There're some pip audit errors that are either for unused dependencies or can be safely ignored. I removed transformers and added ignored vulnerabilities to code_checks.yml

Type of Change

  • 🐛 Bug fix (non-breaking change that fixes an issue)
  • ✨ New feature (non-breaking change that adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 📝 Documentation update
  • 🔧 Refactoring (no functional changes)
  • ⚡ Performance improvement
  • 🧪 Test improvements
  • 🔒 Security fix

Changes Made

  • Removed transformers unused dependency
  • Added vulnerabilities to ignore-vulns in code_checks.yml

Testing

  • uv lock && uv sync --all-extras --dev
  • pip-audit passes (with ignore list)
  • pre-commit run --all-files
  • pytest -m "not integration_test" in aieng-eval-agents
  • smoke imports pass
  • GitHub Actions green on the PR

Manual testing details:

Screenshots/Recordings

Related Issues

Relates to #108

Checklist

  • Code follows the project's style guidelines
  • Self-review of code completed
  • Documentation updated (if applicable)
  • No sensitive information (API keys, credentials) exposed

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.

1 participant