andygrove opened a new pull request, #2789: URL: https://github.com/apache/datafusion-ballista/pull/2789
# Which issue does this PR close? No issue filed. This adds contributor tooling and documentation only. # Rationale for this change Agents working in the repo currently have no entry point to the project docs, and there is no shared guidance for reviewing a PR. A review that only looks at the persona, API and benchmark contracts misses the part reviewers care about most, which is whether the code is correct and tested. # What changes are included in this PR? - Add a short `AGENTS.md` that points to the user guide, the contributor guide (including the user personas page) and the pre-PR steps, and lists the skills under `.ai/skills/`. It links to the docs rather than copying them, so the docs stay the source of truth. - Add a `pr-review` skill at `.ai/skills/pr-review/SKILL.md`. It does a line-level correctness review in context, covering logic, distributed execution, async and concurrency, error handling, resources, config and compatibility, and a test review. On top of that it keeps the Ballista-specific checks for persona red flags, avoidable public API breaks and TPC-H numbers when plan shape or execution changes. - The skill never posts to GitHub unless asked, and drafts review text as one inline comment per issue. # Are there any user-facing changes? No. This only adds files for contributors and agents. There are no code or public API changes. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
