Skip to content

Remove deprecated param from BranchDayOfWeekOperator - #41441

Closed
gopidesupavan wants to merge 2 commits into
apache:mainfrom
gopidesupavan:remove-deprication-param-from-branchdayofweekoperator
Closed

Remove deprecated param from BranchDayOfWeekOperator#41441
gopidesupavan wants to merge 2 commits into
apache:mainfrom
gopidesupavan:remove-deprication-param-from-branchdayofweekoperator

Conversation

@gopidesupavan

Copy link
Copy Markdown
Member

Remove deprication param use_task_execution_day in BranchDayOfWeekOperator


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an 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 newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

@gopidesupavangopidesupavan changed the title Remove depricated from BranchDayOfWeekOperatorRemove depricated param from BranchDayOfWeekOperatorAug 13, 2024

@eladkaleladkal left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Temporary block.

We may want to discuss introducing common.time provider. This will allow to extract several operators/sensor from core and will not put these breaking changes on Airflow 3

@uranusjruranusjr changed the title Remove depricated param from BranchDayOfWeekOperatorRemove deprecated param from BranchDayOfWeekOperatorAug 13, 2024
@romsharon98

Copy link
Copy Markdown
Contributor

Temporary block.

We may want to discuss introducing common.time provider. This will allow to extract several operators/sensor from core and will not put these breaking changes on Airflow 3

I raised thread in mailing list
https://lists.apache.org/thread/2dmlqkcmyomm4q7rrovygs6bw655zx07

@gopidesupavan

Copy link
Copy Markdown
MemberAuthor

Temporary block.
We may want to discuss introducing common.time provider. This will allow to extract several operators/sensor from core and will not put these breaking changes on Airflow 3

I raised thread in mailing list https://lists.apache.org/thread/2dmlqkcmyomm4q7rrovygs6bw655zx07

Thank you @romsharon98 😊 I am happy to help on this work. Please let me know if anything.

@uranusjruranusjr added this to the Airflow 3.0.0 milestone Aug 22, 2024
@github-actions

Copy link
Copy Markdown
Contributor

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 5 days if no further activity occurs. Thank you for your contributions.

@github-actionsgithub-actionsBot added the stale Stale PRs per the .github/workflows/stale.yml policy file label Oct 9, 2024
@gopidesupavan

Copy link
Copy Markdown
MemberAuthor

Closing this pr, as we have moved operators inside standard provider. #41564

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

staleStale PRs per the .github/workflows/stale.yml policy file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@gopidesupavan@romsharon98@eladkal@uranusjr