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

Replace deprecated abc.abstractproperty decorator #13443

Merged
merged 1 commit into from Jan 2, 2021

Conversation

kaxil
Copy link
Member

@kaxil kaxil commented Jan 2, 2021

abc.abstractproperty is deprecated since Python 3.3. Instead we should use @property with abc.abstractmethod.

Docs: https://docs.python.org/3.6/library/abc.html#abc.abstractproperty


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

`abc.abstractproperty` is deprecated since Python 3.3. Instead we should use `@property` with `abc.abstractmethod`.

Docs: https://docs.python.org/3.6/library/abc.html#abc.abstractproperty
@kaxil kaxil force-pushed the 01-remove-deprecated-abc-method branch from b26881f to 399ef88 Compare January 2, 2021 21:09
@kaxil kaxil changed the title Replace deprecated Replace deprecated abc.abstractproperty decorator Jan 2, 2021
@kaxil kaxil requested a review from potiuk January 2, 2021 21:09
@github-actions github-actions bot added the full tests needed We need to run full set of tests for this PR to merge label Jan 2, 2021
@github-actions
Copy link

github-actions bot commented Jan 2, 2021

The PR most likely needs to run full matrix of tests because it modifies parts of the core of Airflow. However, committers might decide to merge it quickly and take the risk. If they don't merge it quickly - please rebase it to the latest master at your convenience, or amend the last commit of the PR, and push it with --force-with-lease.

@kaxil kaxil merged commit d6e1d7b into apache:master Jan 2, 2021
@kaxil kaxil deleted the 01-remove-deprecated-abc-method branch January 2, 2021 22:15
@kaxil kaxil added this to the Airflow 2.0.1 milestone Jan 21, 2021
kaxil added a commit that referenced this pull request Jan 21, 2021
`abc.abstractproperty` is deprecated since Python 3.3. Instead we should use `@property` with `abc.abstractmethod`.

Docs: https://docs.python.org/3.6/library/abc.html#abc.abstractproperty
(cherry picked from commit d6e1d7b)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:logging full tests needed We need to run full set of tests for this PR to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants