Skip to content

Prepare public repository templates and source archives - #6

Merged
tpn merged 3 commits into
mainfrom
codex/013-public-01-prepare-public-repository-templates-and-source-archives
Sep 22, 2026
Merged

tpn merged 3 commits into
mainfrom
codex/013-public-01-prepare-public-repository-templates-and-source-archives

Conversation

@tpn

@tpn tpn commented Sep 22, 2026

Copy link
Copy Markdown
Collaborator

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.

@tpn tpn added the ai-review Request a focused CodeRabbit review label Sep 22, 2026
@tpn
tpn requested a review from melo-gonzo September 22, 2026 05:45
@tpn tpn added ai-review Request a focused CodeRabbit review and removed ai-review Request a focused CodeRabbit review labels Sep 22, 2026
@coderabbitai

coderabbitai Bot commented Sep 22, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

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 configuration

Configuration used: Repository: NVIDIA/cuPhoton/.coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 7aa71c9e-56be-4fa7-bc41-c942a21a78f5

📥 Commits

Reviewing files that changed from the base of the PR and between 201e039 and 4628054.

📒 Files selected for processing (7)
  • .gitattributes
  • .github/CODEOWNERS
  • .github/ISSUE_TEMPLATE/bug.yml
  • .github/ISSUE_TEMPLATE/config.yml
  • .github/ISSUE_TEMPLATE/feature.yml
  • .github/ISSUE_TEMPLATE/task.yml
  • .github/PULL_REQUEST_TEMPLATE.md

Included review availability: Your plan provides up to 12 included reviews per hour; 2 remain after this review.


📝 Walkthrough

Walkthrough

The PR updates export policy, adds structured issue forms and security-report guidance, assigns repository ownership, and adds pull request review requirements.

Changes

Repository contribution controls

Layer / File(s) Summary
Export policy
.gitattributes
Adds export-policy comments, retains GitLab exclusions, and removes GitHub and script export exclusions.
Issue intake forms
.github/ISSUE_TEMPLATE/*
Adds bug, feature, and task forms. Disables blank issues and provides a private security vulnerability submission link.
Contribution review controls
.github/CODEOWNERS, .github/PULL_REQUEST_TEMPLATE.md
Adds a repository-wide ownership rule and pull request checks for validation, repository hygiene, security, documentation, and dependency licenses.

Priority: ⬇️ Low

Merge Risk: ⚪ Minimal · up to 46280

The contribution templates, private security routing, ownership configuration, and export-policy updates are suitable to merge.

🚥 Pre-merge checks | ✅ 2
✅ Passed checks (2 passed)
Check name Status Explanation
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

Comment @coderabbitai help to get the list of available commands.

melo-gonzo
melo-gonzo previously approved these changes Sep 22, 2026

@melo-gonzo melo-gonzo left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Comment thread .github/CODEOWNERS
@tpn

tpn commented Sep 22, 2026

Copy link
Copy Markdown
Collaborator Author

Kept the restored export exclusions and removed only the two broad scripts rules: the MPI rank launcher in #10 is required by the package metadata. The archive check now retains its exact bytes and executable mode, while the other exclusions remain intact.

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.

@tpn
tpn requested a review from melo-gonzo September 22, 2026 17:26

@melo-gonzo melo-gonzo left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Re-approving after the rebase onto main: range-diff against 62e81ef is identical, plus Trent's 96ff94d keeping the MPI launcher in archives.

@tpn
tpn force-pushed the codex/013-public-01-prepare-public-repository-templates-and-source-archives branch from 96ff94d to 1b71a3d Compare September 22, 2026 18:51
tpn and others added 3 commits September 22, 2026 12:03
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>
@tpn
tpn force-pushed the codex/013-public-01-prepare-public-repository-templates-and-source-archives branch from 1b71a3d to 4628054 Compare September 22, 2026 19:05
@tpn

tpn commented Sep 22, 2026

Copy link
Copy Markdown
Collaborator Author

@coderabbitai full review

@coderabbitai

coderabbitai Bot commented Sep 22, 2026 •

Copy link
Copy Markdown
⚠️ Action not completed

Review rate limited.

@tpn

tpn commented Sep 22, 2026

Copy link
Copy Markdown
Collaborator Author

@coderabbitai full review

@coderabbitai

coderabbitai Bot commented Sep 22, 2026 •

Copy link
Copy Markdown
✅ Action performed

Full review finished.

@tpn
tpn merged commit 44c041e into main Sep 22, 2026
12 checks passed
@tpn
tpn deleted the codex/013-public-01-prepare-public-repository-templates-and-source-archives branch September 23, 2026 22:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-review Request a focused CodeRabbit review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants