TheNeuralBit commented on pull request #16522: URL: https://github.com/apache/beam/pull/16522#issuecomment-1026234431
> > > ``` > > > init-hook="from pylint.config import find_pylintrc; import os, sys; sys.path.append(os.path.dirname(find_pylintrc()))" > > > ``` > > Hi @TheNeuralBit. Just commented what I tried to do here to fix it, I thought the `import-error` was due a config issue but I'm not sure it is. I think I'll need some pointers on this because not sure how to solve this error. Ah, I see. I think it's actually probably fine to leave import-error skipped for now. It will be hard to get it right since we have a lot of optional dependencies, and it has limited value, we really should be _exercising_ those dependencies in tests instead which would tease out these issues. -- 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]
