KalleOlaviNiemitalo commented on PR #2906: URL: https://github.com/apache/avro/pull/2906#issuecomment-2110736677
As I understand it, the problem with <https://github.com/apache/avro/pull/2310> is that AVRO-3733 "Migrate package names and artifacts to org.apache.avro namespace" and AVRO-3734 "Integrate build with Apache Avro github actions / local build" have not been done yet. So I would suggest working on those on top of the commits that are in <https://github.com/apache/avro/pull/2310>. It might make sense to merge the current changes from <https://github.com/apache/avro/pull/2310> to a new branch in this repository. That way, a separate pull request for the AVRO-3733 changes could then target that branch and be reviewed here, without looking at the code-drop commits again. Then another PR for AVRO-3734, and finally merge the branch to `main`. OTOH, that would give multiple opportunities to mistakenly use a squash merge and lose the history. -- 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]
