Re: [PR] Standardize deprecations in providers [part1] [airflow]

2024-01-30 Thread via GitHub
mobuchowski merged PR #36876: URL: https://github.com/apache/airflow/pull/36876 -- 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. To unsubscribe, e-mail: commits-unsubscr...@airf

Re: [PR] Standardize deprecations in providers [part1] [airflow]

2024-01-29 Thread via GitHub
dstandish commented on PR #36876: URL: https://github.com/apache/airflow/pull/36876#issuecomment-1915475826 > I am good for it - @dstandish ? What do you think? Seems we just replace warnings.warn with deprecated decorator, seems fine, to me -- This is an automated message from the

Re: [PR] Standardize deprecations in providers [part1] [airflow]

2024-01-25 Thread via GitHub
potiuk commented on PR #36876: URL: https://github.com/apache/airflow/pull/36876#issuecomment-1910568926 I am good for it - @dstandish ? What do you think? -- 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

Re: [PR] Standardize deprecations in providers [part1] [airflow]

2024-01-24 Thread via GitHub
kacpermuda commented on PR #36876: URL: https://github.com/apache/airflow/pull/36876#issuecomment-1908197262 Hey, is there anything else that i can do to make this PR merged? Pinging those who at least saw it 😄 @eladkal @potiuk @o-nikolas -- This is an automated message from the Apache

Re: [PR] Standardize deprecations in providers [part1] [airflow]

2024-01-22 Thread via GitHub
kacpermuda commented on PR #36876: URL: https://github.com/apache/airflow/pull/36876#issuecomment-1904338840 @potiuk I already prepared a POC for the decorator itself in #36952 . @dstandish check it out, maybe it will help detecting the things that should be removed in specific versions of

Re: [PR] Standardize deprecations in providers [part1] [airflow]

2024-01-19 Thread via GitHub
o-nikolas commented on PR #36876: URL: https://github.com/apache/airflow/pull/36876#issuecomment-1900945241 I don't have time to look through this large PR in detail at the moment. But I love the goals of documentation and standardization! +1 -- This is an automated message from the Apach