Skip to content

Improve SANO sample#495

Merged
Quinn-With-Two-Ns merged 3 commits into
mainfrom
improve-sano-sample
Jul 13, 2026
Merged

Improve SANO sample#495
Quinn-With-Two-Ns merged 3 commits into
mainfrom
improve-sano-sample

Conversation

@Quinn-With-Two-Ns

@Quinn-With-Two-Ns Quinn-With-Two-Ns commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

What was changed

Improve SANO sample

Why?

Checklist

  1. Closes

  2. How was this tested:

  1. Any docs updates needed?

@Quinn-With-Two-Ns
Quinn-With-Two-Ns requested a review from a team as a code owner July 13, 2026 21:29
bergundy
bergundy approved these changes Jul 13, 2026
View reviewed changes

@bergundy bergundy left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I would make the default (fallback) namespace my-caller-namespace or my-handler-namespace instead of default but otherwise LGTM.

@Quinn-With-Two-Ns
Quinn-With-Two-Ns merged commit 87f8c9b into main Jul 13, 2026
13 checks passed
@Quinn-With-Two-Ns
Quinn-With-Two-Ns deleted the improve-sano-sample branch July 13, 2026 23:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bergundy bergundy bergundy approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Quinn-With-Two-Ns @bergundy