tisonkun commented on code in PR #59: URL: https://github.com/apache/steve/pull/59#discussion_r2488529320
########## v3/server/bin/asf-load-ldap.py: ########## @@ -29,7 +29,7 @@ DB_FNAME = THIS_DIR.parent / 'steve.db' sys.path.insert(0, str(THIS_DIR.parent.parent)) -import steve.persondb +import steve.persondb # noqa: E402 Review Comment: > E402 Module level import not at top of file This is a temporary suppression and IMO we can move the import once we live with `uv run` for all dev workflow, which can properly resolve PYTHONPATH and we don't need the `sys.path.insert(0, str(THIS_DIR.parent.parent))` trick then. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
