Skip to content

Add Dusty and Sam Julien as code owners - #857

Merged
davidmckayv merged 2 commits into
mainfrom
david/codeowners-dusty-sam
Oct 28, 2026
Merged

davidmckayv merged 2 commits into
mainfrom
david/codeowners-dusty-sam

Conversation

@davidmckayv

Copy link
Copy Markdown
Contributor

Adds @dustywusty and @samjulien to .github/CODEOWNERS, so either can approve a change. Both are now repository admins and on the list of accounts that may merge to main, which the file says go together.

Anthropic's SDK 2.3.0, which CI now installs for the AG2 harness, sends the
key as Authorization: Bearer instead of x-api-key. The agent's key still
reaches Anthropic; the test only looked for the old header, so it failed on
every pull request. It now accepts either, and passes with Anthropic's SDK
1.17.0 and 2.3.0 alike.

(cherry picked from commit a1391e7)

@guidovizoso guidovizoso 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. Both are admins on the repo, so the entry is valid.

@davidmckayv
davidmckayv merged commit 50ba17a into main Oct 28, 2026
19 checks passed
@davidmckayv
davidmckayv deleted the david/codeowners-dusty-sam branch October 28, 2026 02:12
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