tfx-addons icon indicating copy to clipboard operation
tfx-addons copied to clipboard

Align pylintrc with the one used by TFX

Open TheMichaelHu opened this issue 3 years ago • 2 comments

In #63 we needed to modify the original pylintrc to ignore super-with-arguments in order to get the sklearn examples to pass pre-commit checks.

Fix the linter errors in the sklearn example and remove the change that was added to pylint configs.

TheMichaelHu avatar Aug 09 '21 14:08 TheMichaelHu

@casassg FYI for comment

rcrowe-google avatar Aug 13 '21 16:08 rcrowe-google

yup. We talked about this in #63

That said, super-with-arguments is not a terrible thing to have either, so I would be fine by keeping it (actually TFX does use some of those as well it seems)

casassg avatar Aug 13 '21 16:08 casassg