yangzhang75 commented on PR #8441: URL: https://github.com/apache/texera/pull/8441#issuecomment-5591789219
Thanks, addressed. Since that comment I force-pushed additional tests (head is now `48d89d3`): the terminal-always display path, the disabled-operator and disabled-downstream-link cases, the un-curated terminal default, and the live picker rebuild on an eye toggle. Measuring the current patch locally with the specs that exercise the changed source files (`workflow-form.component.ts` and `workflow-result.service.ts`), the changed lines are now at 100% statement and function coverage, 0 missing. The Codecov comment above still reads 80.6% because it is pinned to my first commit `89e7b52` and has not recomputed on `48d89d3` yet: the fork-PR workflows are still queued (combined status pending, no test job has run), so Codecov has nothing new to report. It should refresh once CI runs on the new commit. Could a maintainer approve the workflow run so it recomputes? -- 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]
