amoghrajesh commented on PR #32495: URL: https://github.com/apache/airflow/pull/32495#issuecomment-1631832416
> > One more fix needed. There is a static check failing with missing "jsonschema" and for a very good reason. > > Here is one special thing to fix that is not obvious. You need make the `jsonschema` imports local and not at the top of the file, The main reason is `click`. Long story short - we need to have very minimal set of dependencies needed to "parse" all files that contain `@click` annotation, because effectively those command files are parsed every time when `<TAB>` is used to autocomplete Breeze (and this is done witthout entering breeze's virtualenv) - so error here indicates, that you need to make sure that jsonschema package is imported locally when needed and not at the top of the files that are imported by any of the breeze commands. Wow this is interesting. Was not aware of this behaviour. I have made the changes here accordingly -- 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]
