Re: [PR] Add docstring to airflow/configurations.py to avoid D105 [airflow]

2024-02-20 Thread via GitHub
romsharon98 commented on code in PR #37540: URL: https://github.com/apache/airflow/pull/37540#discussion_r1495441813 ## airflow/configuration.py: ## @@ -1883,8 +1884,9 @@ def __getstate__(self): ] } -def __setstate__(self, state): -self.__init

Re: [PR] Add docstring to airflow/configurations.py to avoid D105 [airflow]

2024-02-19 Thread via GitHub
Taragolis commented on code in PR #37540: URL: https://github.com/apache/airflow/pull/37540#discussion_r1495061759 ## airflow/configuration.py: ## @@ -1883,8 +1884,9 @@ def __getstate__(self): ] } -def __setstate__(self, state): -self.__init__

Re: [PR] Add docstring to airflow/configurations.py to avoid D105 [airflow]

2024-02-19 Thread via GitHub
romsharon98 commented on code in PR #37540: URL: https://github.com/apache/airflow/pull/37540#discussion_r1495026409 ## airflow/configuration.py: ## @@ -1883,8 +1884,9 @@ def __getstate__(self): ] } -def __setstate__(self, state): -self.__init

Re: [PR] Add docstring to airflow/configurations.py to avoid D105 [airflow]

2024-02-19 Thread via GitHub
romsharon98 commented on code in PR #37540: URL: https://github.com/apache/airflow/pull/37540#discussion_r1495025369 ## airflow/configuration.py: ## @@ -1883,8 +1889,19 @@ def __getstate__(self): ] } -def __setstate__(self, state): -self.__ini

Re: [PR] Add docstring to airflow/configurations.py to avoid D105 [airflow]

2024-02-19 Thread via GitHub
romsharon98 commented on code in PR #37540: URL: https://github.com/apache/airflow/pull/37540#discussion_r1495025258 ## airflow/configuration.py: ## @@ -1871,7 +1871,13 @@ def _warn_deprecate( stacklevel=4 + extra_stacklevel, ) -def __getstate

Re: [PR] Add docstring to airflow/configurations.py to avoid D105 [airflow]

2024-02-19 Thread via GitHub
Taragolis commented on code in PR #37540: URL: https://github.com/apache/airflow/pull/37540#discussion_r1494396180 ## airflow/configuration.py: ## @@ -1883,8 +1889,19 @@ def __getstate__(self): ] } -def __setstate__(self, state): -self.__init_

[PR] Add docstring to airflow/configurations.py to avoid D105 [airflow]

2024-02-19 Thread via GitHub
romsharon98 opened a new pull request, #37540: URL: https://github.com/apache/airflow/pull/37540 related: https://github.com/apache/airflow/issues/37523 --- **^ Add meaningful description above** Read the **[Pull Request Guidelines](https://github.com/apa