iajoiner commented on PR #3: URL: https://github.com/apache/arrow-ballista-python/pull/3#issuecomment-1416916099
@andygrove There are several remaining issues that need to be addressed: 1. Now that ballista-python is in a different repo compared to ballista I wonder how versioning of the former will change. Shall ballista-python be dependent on released versions of ballista now? 2. Some Rust code doesn't pass` cargo fmt --all -- --check`. I'm not sure why that's the case and whether I should file another PR just to fix the formatting. 3. Python source files in `docs`, `dev` and `ci` have never been subject to flake8 before since we used to restrict it to the old python directory which actually contains no python scripts at all and hence are often non-compliant. I added them to `.flake8` for now but later we need to get these files compliant as well which I will fix. -- 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]
