cswartzvi opened a new pull request, #1371: URL: https://github.com/apache/hamilton/pull/1371
Hi all - I was looking through the codebase to help create the first ASF release (#1336) and thought we might benefit from a little clean-up. Still a work in progress, feel free to critique any of my changes. ## Changes - [x] Moved top-level images to new `assets` folder. Do we need to keep these? - [x] Move top-level md files to either `docs` or `writeups`. Maybe these should be fully integrated with the docs? - [x] Delete `setup.py`, removing the legacy Python packaging script and related logic - [x] Remove unused configuration files (`.style.yapf`, `.flake8`, ...) - [x] Update manifest - [ ] Fix `ruff` configuration - [ ] Update `uv` configuration - [ ] Fix broken tests (maybe out of scope for this PR) ## How I tested this No changes to codebase ## Notes Nothing yet ## Checklist - [x] PR has an informative and human-readable title (this will be pulled into the release notes) - [x] Changes are limited to a single goal (no scope creep) - [ ] Code passed the pre-commit check & code is left cleaner/nicer than when first encountered. - [ ] Any _change_ in functionality is tested - [ ] New functions are documented (with a description, list of inputs, and expected output) - [ ] Placeholder code is flagged / future TODOs are captured in comments - [ ] Project documentation has been updated if adding/changing functionality. -- 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]
