gopidesupavan commented on code in PR #46829: URL: https://github.com/apache/airflow/pull/46829#discussion_r1958420368
########## contributing-docs/11_provider_packages.rst: ########## @@ -380,17 +380,11 @@ generous policy of supporting multiple versions of providers at the same time. A backward compatible with future versions of Airflow, so you can upgrade Airflow and keep the providers at the same version. -.. note:: - - In the new structure of providers the CHANGELOG.rst is named CHANGELOG.rst and it is present in - the docs subfolder of the provider. TODO(potiuk) replace references to CHANGELOG.rst when we - move all providers to the new structure. - When you introduce a breaking change in the provider, you have to make sure that you communicate it -properly. You have to update ``CHANGELOG.rst`` file in the provider package. Ideally you should provide -a migration path for the users to follow in the``CHANGELOG.rst``. +properly. You have to update ``changelog.rst`` file in the ``docs`` folder of the provider package. +Ideally you should providea migration path for the users to follow in the``changelog.rst``. Review Comment: ```suggestion Ideally you should provide a migration path for the users to follow in the``changelog.rst``. ``` -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
