mengw15 commented on PR #5164: URL: https://github.com/apache/texera/pull/5164#issuecomment-4527706733
A general question about Codecov policy: I understand that we want to aim for getting every PR to 100% patch coverage. But sometimes covering the last few branches requires changes that go well beyond the PR's actual scope — for example, refactoring production code purely for testability (introducing DI seams or test-only constructor parameters), or adopting a new project-wide testing framework (e.g. a mocking library the project doesn't currently use). Each of those is a project-wide trade-off, not a PR-local decision. Question: is 100% Codecov patch coverage a hard merge requirement? -- 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]
