looker-open-source / henry

A command line tool for Looker instance cleanup
MIT License
79 stars 27 forks source link

Update python and sdk versions #54

Closed tpakeman closed 2 years ago

tpakeman commented 2 years ago

NOTE: this hardcodes the requirement to use Looker SDK 4.0 due to the structure of the new looker_sdk libraries. Methods are called as serialize40, imports come from looker_sdk.sdk.api40 and in two cases the string literal 4.0 is passed into an object constructor

tpakeman commented 2 years ago

Resolved all changes, except in conftest.py we do need to keep aliasing looker_sdk as client to avoid conflict with a method name in that file

tpakeman commented 2 years ago

I've run the black formatter with the default settings, but now there are additional formatting changes. Can you confirm if you have a custom configuration for the formatter?