Ma77Ball commented on PR #5124:
URL: https://github.com/apache/texera/pull/5124#issuecomment-4481261200
Hi @PG1204, while I begin my review, please address @Yicong-Huang's
feedback. Specifically:
1. Update the PR description to follow this template exactly:
```
### What changes were proposed in this PR?
...
### Any related issues, documentation, or discussions?
...
### How was this PR tested?
...
### Was this PR authored or co-authored using generative AI tooling?
...
```
2. Add test coverage for as much of the new code as possible. At a minimum,
please cover the main features and call paths introduced here.
3. Relocate the overall PR plan to the parent issue, and keep this PR's
description scoped to the code changes it actually contains.
4. Document any architectural changes. If this PR modifies the architecture,
please describe what changed and where, so reviewers can follow the design
intent.
Thanks, and looking forward to the updates!
--
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]
