Abhishekmishra2808 commented on PR #608: URL: https://github.com/apache/tooling-trusted-releases/pull/608#issuecomment-3818539449
Hi @sbp , ### PR Final Update I have completed a surgical refactor to address all review feedback and align with project conventions: * **Import Compliance**: Refactored imports to follow the "least significant name part" convention and renamed variables to `msg` to avoid conflicts. * **Surgical Logic**: Reverted style-only changes (like `.encode` vs `bytes`) and restored original method order and comments. * **Dependency Fix**: Manually reverted `uv.lock` to restore `alembic` to version `1.18.1`, keeping only the mandatory CI timestamp. * **UTF-8 Support**: Implemented `policy.SMTPUTF8` and added `test_send_handles_non_ascii_headers` to verify international header support. * **Docstring Cleanup**: Removed subjective "gold-standard" phrasing and meta-commentary regarding maintainers from all files. All 12 unit tests are passing. Thank you for the guidance on these project standards. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
