youssef-itanii commented on issue #1051: URL: https://github.com/apache/iceberg-python/issues/1051#issuecomment-2286857469
> Hi @youssef-itanii and @Minfante377 - thank you again for working on these PRs, and sorry for this unfortunate concurrency conflict. I think we could avoid a concurrency conflict in the future by first assigning a story to a potential solution contributor. > > For this specific issue, I think it would make sense to move forward with @Minfante377 's PR as I think having a make command to run both actions is a good idea: > > ``` > test-coverage: | test-coverage-unit test-coverage-integration > poetry run coverage combine .coverage.unit .coverage.integration > ``` No problem! It was quite a learning experience :smile: I'll make sure to self assign as well in the future! -- 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: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
