Skip to content

Remove template plugin from public catalog#45

Merged
eric-tramel merged 1 commit intomainfrom
codex/remove-template-from-public-catalog
May 8, 2026
Merged

Remove template plugin from public catalog#45
eric-tramel merged 1 commit intomainfrom
codex/remove-template-from-public-catalog

Conversation

@eric-tramel
Copy link
Copy Markdown
Contributor

What

Removes data-designer-template from the checked-in public catalog at catalog/plugins.json.

The template package remains in the repository as the reference implementation and scaffold example, but it is no longer advertised as an installable package through the published NVIDIA catalog.

Why

Now that catalog entries are registered explicitly for first release, the reference template should not stay in the public catalog. It is documentation and authoring material, not a package we intend users to discover and install from the catalog.

Usage

No user-facing command changes. After this deploys, the published catalog should include only release-intended packages:

data-designer-github
data-designer-retrieval-sdg

How

  • Removed the data-designer-template package object from catalog/plugins.json.
  • Added a checked-in catalog assertion that the template package is excluded.
  • Updated release-validation coverage so the template plugin is expected not to be publicly cataloged.
  • Changed the release Makefile usage comment to use a non-template package name.

Validation

  • make lint
  • make test-devtools
  • make check
  • make docs

@eric-tramel eric-tramel marked this pull request as ready for review May 8, 2026 21:03
@eric-tramel eric-tramel requested a review from a team as a code owner May 8, 2026 21:03
Copy link
Copy Markdown
Contributor

@johnnygreco johnnygreco left a comment

Choose a reason for hiding this comment

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

Reviewed the public catalog update, release-validation test change, and Makefile example. Local validation is green (make lint, make test-devtools, make check, make docs), and the PR checks are passing. Looks good to me.

@eric-tramel eric-tramel merged commit 9910bfe into main May 8, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants