Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

refactor: api v2 breaking changes CI check#24999

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Conversation

@supalarry
Copy link
Contributor

@supalarrysupalarry commentedNov 7, 2025
edited by cubic-dev-aibot
Loading

Fixes#24996


Summary by cubic

Refactor the API v2 CI breaking-changes check to use tufin/oasdiff via Docker and an ignore list, making the check more reliable and reducing noise from known errors.

  • Refactors
    • Replace oasdiff GitHub Action with a Docker command.
    • Add .github/oasdiff-err-ignore.txt to whitelist expected ERRs.
    • Preserve fail-on ERR behavior and existing Swagger generation/tests.

Written for commit655d6d5. Summary will update automatically on new commits.

@linear
Copy link

linearbot commentedNov 7, 2025

@github-actionsgithub-actionsbot added the 🚨 urgentCreated by Linear-GitHub Sync labelNov 7, 2025
@graphite-appgraphite-appbot requested a review froma teamNovember 7, 2025 14:13
@keithwillcodekeithwillcode added corearea: core, team members only platformAnything related to our platform plan labelsNov 7, 2025
@supalarrysupalarryenabled auto-merge (squash)November 7, 2025 14:15
Copy link
Contributor

@cubic-dev-aicubic-dev-aibot left a comment

Choose a reason for hiding this comment

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

No issues found across 2 files

@vercel
Copy link

vercelbot commentedNov 7, 2025
edited
Loading

The latest updates on your projects. Learn more aboutVercel for GitHub.

2 Skipped Deployments
ProjectDeploymentPreviewCommentsUpdated (UTC)
calIgnoredIgnoredNov 7, 2025 2:36pm
cal-euIgnoredIgnoredNov 7, 2025 2:36pm

@supalarrysupalarry merged commit799e354 intomainNov 7, 2025
38 checks passed
@supalarrysupalarry deleted the lauris/cal-6714-fix-whitelist-oasdiff-breaking-changes branchNovember 7, 2025 15:13
@github-actions
Copy link
Contributor

E2E results are ready!

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

Reviewers

@cubic-dev-aicubic-dev-ai[bot]cubic-dev-ai[bot] left review comments

@ThyMinimalDevThyMinimalDevThyMinimalDev approved these changes

Assignees

No one assigned

Labels

corearea: core, team members onlyplatformAnything related to our platform planready-for-e2esize/S🚨 urgentCreated by Linear-GitHub Sync

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

fix: whitelist oasdiff breaking changes

4 participants

@supalarry@ThyMinimalDev@keithwillcode

[8]ページ先頭

©2009-2025 Movatter.jp