Skip to content

Add kernel_name to examples.json#2903

Merged
paoloredis merged 3 commits intomainfrom
DOC-6376
Mar 18, 2026
Merged

Add kernel_name to examples.json#2903
paoloredis merged 3 commits intomainfrom
DOC-6376

Conversation

@paoloredis
Copy link
Collaborator

@paoloredis paoloredis commented Mar 18, 2026

Note

Low Risk
Low risk: adds an optional kernelName field derived from a new KERNEL_NAME marker without changing existing example parsing behavior or execution flow.

Overview
Adds support for an optional per-example kernel identifier. The Example parser now recognizes a KERNEL_NAME marker in example source files and stores it as kernel_name, and both the remote (component.py) and local (local_examples.py) example builders emit this value as kernelName in generated example metadata when present.

Written by Cursor Bugbot for commit b2cef51. This will update automatically on new commits. Configure here.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 18, 2026

DOC-6376

@jit-ci
Copy link

jit-ci bot commented Mar 18, 2026

🛡️ Jit Security Scan Results

CRITICAL HIGH MEDIUM

✅ No security findings were detected in this PR


Security scan by Jit

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Copy link
Contributor

@andy-stark-redis andy-stark-redis left a comment

Choose a reason for hiding this comment

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

Yeah, I got kernelName to appear in examples.json for a local build, so LGTM.

@paoloredis paoloredis merged commit 798255d into main Mar 18, 2026
46 checks passed
@paoloredis paoloredis deleted the DOC-6376 branch March 18, 2026 11:32
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