ibzib commented on pull request #13668: URL: https://github.com/apache/beam/pull/13668#issuecomment-761072772
> @sonam-vend and I were thinking we'd likely have only a small-ish number of issues (hopefully zero or thereabouts!) when we enable the switch at the global level. @shehzaadn-vd Why hope for zero issues when we could guarantee zero issues? > Since eventually we'll be turning on the global switch anyway, enabling strict dependencies at the module level would have to be undone anyway. It's a trivial amount of work to add and remove these settings. The benefit of gaining strict dependency analysis on each module immediately outweighs the cost of copy+paste+deleting a few lines in each file. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
