litellm icon indicating copy to clipboard operation
litellm copied to clipboard

[Feature]: Add linting / code quality check to ensure `load_dotenv()` is not called

Open ishaan-jaff opened this issue 1 year ago • 1 comments

The Feature

we've had 1-2 issues regarding not calling load_dotenv(). We should ensure that we don't add code in the future that calls it. It would really help to have some linting / code quality checks around this

https://github.com/BerriAI/litellm/issues/4888

Motivation, pitch

Twitter / LinkedIn details

No response

ishaan-jaff avatar Jul 26 '24 02:07 ishaan-jaff

@ishaan-jaff Hi!

Regarding the functionality this issue aims to achieve, it was something I also wanted and thought there would be demand for, so I've created a Flake8 plugin to address it.

https://github.com/K-dash/flake8-import-guard (Please refer to the README.md for detailed functionality)

If my plugin meets your requirements, would you consider incorporating it into this project? Also, if you have any questions, please feel free to ask!

K-dash avatar Jul 31 '24 02:07 K-dash

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

github-actions[bot] avatar Jan 28 '25 02:01 github-actions[bot]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

github-actions[bot] avatar Apr 29 '25 00:04 github-actions[bot]