Conversation
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository: NVIDIA/cuPhoton/.coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (7)
Included review availability: Your plan provides up to 12 included reviews per hour; 2 remain after this review. 📝 WalkthroughWalkthroughThe PR updates export policy, adds structured issue forms and security-report guidance, assigns repository ownership, and adds pull request review requirements. ChangesRepository contribution controls
Priority: ⬇️ Low Merge Risk: ⚪ Minimal · up to The contribution templates, private security routing, ownership configuration, and export-policy updates are suitable to merge. 🚥 Pre-merge checks | ✅ 2✅ Passed checks (2 passed)
Comment |
melo-gonzo
left a comment
There was a problem hiding this comment.
LGTM. I pushed 62e81ef restoring the internal export-ignore block (.gitlab, .gitlab-ci.yml, scripts, AGENTS.md, CLAUDE.md, CHANGELOG.md, RELEASING.md) so .gitattributes matches the internal tree and does not conflict on the next promotion. The only intended difference from the previous public file is the removal of the .github lines.
|
Kept the restored export exclusions and removed only the two broad A second maintainer would help with review coverage. The current rules require independent approval but do not require CODEOWNERS approval, so I’ve left owner selection for a separate maintainership change. |
96ff94d to
1b71a3d
Compare
Add issue and pull request templates and repository ownership metadata. Keep all tracked public runtime files available in source archives. Signed-off-by: Trent Nelson <trentn@nvidia.com>
Signed-off-by: Carmelo Gonzales <carmelog@nvidia.com>
Retain the public rank launcher required by source-archive builds. Keep the remaining development and release-file exclusions intact. Signed-off-by: Trent Nelson <trentn@nvidia.com>
1b71a3d to
4628054
Compare
|
@coderabbitai full review |
|
|
@coderabbitai full review |
✅ Action performedFull review finished. |
This PR adds issue and pull request templates and CODEOWNERS entries so contributors have a consistent way to report problems, propose changes, and find reviewers. It also removes archive exclusions so source archives contain the tracked runtime files needed to build and use cuPhoton.