search

Not-Diamond / notdiamond-python

Not Diamond Python SDK
54 stars 8 forks source link

Removing NLTK due to security vulnerability. #8

Closed acompa closed 4 months ago

acompa commented 4 months ago

Addressing https://github.com/Not-Diamond/notdiamond-python/security/dependabot/6 by removing dependencies on NLTK and llama-index. These are used in a skipped test.

While NLTK maintainers are working towards a fix per nltk/nltk#3266, I'd like to cover this issue asap.

github-actions[bot] commented 4 months ago

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)		Lines missing
  notdiamond/toolkit
  custom_router.py
  tests/test_documentation
  test_structured_output.py
Project Total  

This report was generated by python-coverage-comment-action