Skip to content
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

Install cattr on Python 3.7 - Fix docs build on RTD #12045

Merged
merged 1 commit into from
Nov 2, 2020

Conversation

mik-laj
Copy link
Member

@mik-laj mik-laj commented Nov 2, 2020

I believe this library is still needed for Python 3.7, but the constraints currently preclude installing this library on this version of Python.
This causes a problem on RTD.
https://readthedocs.org/projects/airflow/builds/12243425/

Related: #11969 CC: @kaxil


^ 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 UPDATING.md.

@mik-laj mik-laj requested review from ashb, potiuk and kaxil November 2, 2020 15:40
@kaxil kaxil merged commit fabcb2d into apache:master Nov 2, 2020
@kaxil kaxil added this to the Airflow 1.10.13 milestone Nov 2, 2020
kaxil pushed a commit to astronomer/airflow that referenced this pull request Nov 5, 2020
potiuk pushed a commit that referenced this pull request Nov 15, 2020
potiuk pushed a commit that referenced this pull request Nov 16, 2020
potiuk pushed a commit that referenced this pull request Nov 16, 2020
kaxil pushed a commit that referenced this pull request Nov 18, 2020
kaxil pushed a commit to astronomer/airflow that referenced this pull request Nov 27, 2020
(cherry picked from commit fabcb2d)
(cherry picked from commit 707d762)
cfei18 pushed a commit to cfei18/incubator-airflow that referenced this pull request Mar 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants