Skip to content

feat: adds support for self in overlay#322

Open
baywet wants to merge 7 commits intoOAI:mainfrom
baywet:feat/self-support
Open

feat: adds support for self in overlay#322
baywet wants to merge 7 commits intoOAI:mainfrom
baywet:feat/self-support

Conversation

@baywet
Copy link
Copy Markdown
Member

@baywet baywet commented Apr 13, 2026

fixes #310

heavy use of GCC to put this one together. (based on the discussion in the issue, and the arazzo PR as a template)

Signed-off-by: Vincent Biret <vincentbiret@hotmail.com>
@baywet baywet self-assigned this Apr 13, 2026
Comment thread versions/1.2.0-dev.md Outdated
Comment thread versions/1.2.0-dev.md
Comment thread versions/1.2.0-dev.md Outdated
Comment thread versions/1.2.0-dev.md Outdated
Comment thread versions/1.2.0-dev.md Outdated
Comment thread versions/1.2.0-dev.md Outdated
Comment thread versions/1.2.0-dev.md Outdated
Comment thread versions/1.2.0-dev.md Outdated
Comment thread versions/1.2.0-dev.md Outdated
Comment thread versions/1.2.0-dev.md
Comment thread versions/1.2.0-dev.md Outdated
Comment thread versions/1.2.0-dev.md Outdated
Signed-off-by: Vincent Biret <vincentbiret@hotmail.com>
@baywet baywet requested a review from handrews April 14, 2026 13:55
Comment thread versions/1.2.0-dev.md
baywet added 3 commits April 15, 2026 12:53
Signed-off-by: Vincent Biret <vincentbiret@hotmail.com>
Signed-off-by: Vincent Biret <vincentbiret@hotmail.com>
@baywet baywet requested a review from handrews April 15, 2026 17:04
@baywet baywet marked this pull request as ready for review April 21, 2026 15:15
@baywet baywet requested a review from a team as a code owner April 21, 2026 15:15
Copy link
Copy Markdown
Member

@handrews handrews left a comment

Choose a reason for hiding this comment

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

This is getting quite close! BTW, I have only had time to review the markdown file, I hope someone else can look at the tests and schemas, and just ping me if there are questions you can't resolve.

Comment thread versions/1.2.0-dev.md Outdated
Comment thread versions/1.2.0-dev.md Outdated
Comment thread versions/1.2.0-dev.md Outdated
Comment thread versions/1.2.0-dev.md
Co-authored-by: Vincent Biret <vincentbiret@hotmail.com>
Comment thread versions/1.2.0-dev.md Outdated
Comment thread versions/1.2.0-dev.md Outdated
@baywet baywet requested a review from handrews April 29, 2026 10:53
Copy link
Copy Markdown
Member

@handrews handrews left a comment

Choose a reason for hiding this comment

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

GitHub is acting weird- it still shows my "sorry I didn't hit send two days ago" as not having been sent despite it obviously having been.

More importantly, it's showing the entire Markdown file as deleted and re-added?

I'm pretty sure tha changes are correct now based on the discussion, but the diff is unusable and I'm worried something went wrong with a recent commit?

@baywet baywet force-pushed the feat/self-support branch from e789604 to 6f3adf7 Compare April 29, 2026 16:01
@baywet
Copy link
Copy Markdown
Member Author

baywet commented Apr 29, 2026

More importantly, it's showing the entire Markdown file as deleted and re-added?

It looks like the commit I had made from the UI messed up the line returns somehow. I reapplied that commit correctly and force-pushed. The diff looks ok now. Thanks for calling it out!

Copy link
Copy Markdown
Member

@handrews handrews left a comment

Choose a reason for hiding this comment

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

SHIP IT! 😁

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.

The extends field should support referencing OADs by their $self

2 participants