blag commented on code in PR #24590: URL: https://github.com/apache/airflow/pull/24590#discussion_r904135278
########## airflow/cli/__init__.py: ########## @@ -15,3 +15,108 @@ # KIND, either express or implied. See the License for the # specific language governing permissions and limitations # under the License. +import os + +import rich_click as click Review Comment: That looks like a reasonable suggestion, but that import comes [straight out of the rich_click documentation](https://github.com/ewels/rich-click#import-as-click): > ## Usage ## > ### Import as click ### > > To use rich-click, switch out your normal click import with rich-click, using the same namespace: > > ```python > import rich_click as click > ``` > > That's it ✨ Then continue to use click as you would normally. The rich_click package tries to be a superset of Click. When people are trying to add additional options or commands, I am trying to direct them to the Click documentation, since that is what's relevant to them. If that justification doesn't sway you, I'm happy to change it if you insist (just confirm in a comment below). -- 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]
