New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update PythonVirtualenvOperator Howto #24782
Conversation
Congratulations on your first Pull Request and welcome to the Apache Airflow community! If you have any issues or are unsure about any anything please check our Contribution Guide (https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst)
|
We should mention |
It is mentioned in the change, do you suggest to mention in another part of the documentation as well? |
Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
70caa5f
to
d391d93
Compare
Awesome work, congrats on your first merged pull request! |
* Update PythonVirtualenvOperator Howto (cherry picked from commit 011fbae)
Currently, there is no hint in the documentation that the
virtualenv
package needs to be installed in the host environment when usingPythonVirtualenvOperator
. There is an exception raised when it is not the case but I would have found it good to find the requirement already in its documentation. This PR brings a description as a change.^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code change, Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragement file, named
{pr_number}.significant.rst
, in newsfragments.