Skip to content

dev: sync with main - #5493

Merged
handrews merged 7 commits into
devfrom
dev-sync-with-main
Aug 13, 2026
Merged

dev: sync with main#5493
handrews merged 7 commits into
devfrom
dev-sync-with-main

Conversation

@oai-spec-publisher

Copy link
Copy Markdown
Contributor

Merge relevant changes from main into dev.

baywet and others added 3 commits August 13, 2026 13:17
Signed-off-by: Vincent Biret <vincentbiret@hotmail.com>
ci: fixes sync script to avoid grabbing old js files
Signed-off-by: Vincent Biret <vincentbiret@hotmail.com>
@handrews

Copy link
Copy Markdown
Member

I'm not sure this is right after all. Let me check a few things.

@handrews

Copy link
Copy Markdown
Member

OK the : prefix thing is not working for me locally, @baywet is that documented somewhere?

Also, I've found some other issues with these tests and their configuration across the branches (none of them are quite the same, and I don't think that's always intentional). I will file an issue later today once I have a chance to write it up. I'm leaving this up in draft for now.

@baywet

baywet commented Aug 13, 2026

Copy link
Copy Markdown
Member

@handrews see #5494

baywet and others added 3 commits August 13, 2026 15:37
Co-authored-by: Vincent Biret <vincentbiret@hotmail.com>
ci: negates path spec and neutralizes non-zero exit codes
@handrews

Copy link
Copy Markdown
Member

@baywet this is still not picking up the change to tests/schema/oas-schema.mjs. Do we need to add a changing commit to it? I'm guessing that since it was previously merged and then deleted, it will need a new commit.

@baywet

baywet commented Aug 13, 2026

Copy link
Copy Markdown
Member

@handrews I didn't trace the history but I suspect since there was a merge and revert commit (from reading the script) it's not going to pick it up unless we either edit the base file (in main) or manually change it in dev

@handrews
handrews marked this pull request as ready for review August 13, 2026 22:00
@handrews
handrews merged commit dbb9f27 into dev Aug 13, 2026
2 checks passed
@handrews
handrews deleted the dev-sync-with-main branch August 13, 2026 22:00
@handrews

Copy link
Copy Markdown
Member

I went ahead and merged this so we will have it in place when PR #5499 merges. I'll go tack a trivial commit on the other file to see if we can get it picked up.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants