potiuk commented on a change in pull request #12082:
URL: https://github.com/apache/airflow/pull/12082#discussion_r518725211



##########
File path: scripts/ci/docker-compose/local.yml
##########
@@ -48,6 +48,7 @@ services:
       - ../../../hooks:/opt/airflow/hooks:cached
       - ../../../logs:/root/airflow/logs:cached
       - ../../../pylintrc:/opt/airflow/pylintrc:cached
+      - ../../../pyproject.toml:/opt/airflow/pyproject.toml:cached

Review comment:
       Needed for black formatting of generated setup.py files.

##########
File path: dev/provider_packages/README.md
##########
@@ -171,3 +171,124 @@ the version in PyPI does not contain the leading 0s in 
version name - therefore
 also do not container the leading 0s.
 
 * You can install the .whl packages with `pip install <PACKAGE_FILE>`
+

Review comment:
       I've added document about manual procedure for testing if the 
provider/backport packges are importable. That saves a lot of time for testing 
of those scripts because you can initialize the whole container with all the 
deps once and tests the scripts inside.




----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to