Skip to content
This repository was archived by the owner on Mar 5, 2026. It is now read-only.

Migrate ignore/test#2638

Merged
mkondratek merged 2 commits intomainfrom
mkondratek/chore/migrate-api-part-4
Nov 14, 2024
Merged

Migrate ignore/test#2638
mkondratek merged 2 commits intomainfrom
mkondratek/chore/migrate-api-part-4

Conversation

@mkondratek
Copy link
Copy Markdown
Contributor

@mkondratek mkondratek commented Nov 13, 2024

This PR is a part of the protocol migration. Some endpoints are written by hand. We are switching to the protocol generated from Cody. In this PR:

Based on PR #2637

Full chain of PRs as of 2024-11-13

Test plan

  • All migrated endpoints have been verified with debugger (and reviewed in the trace log).
  • Cody Ignore (with CODY_JETBRAINS_FEATURES=cody.feature.internals-menu=true) 🟢

Copy link
Copy Markdown
Contributor

@pkukielka pkukielka left a comment

Choose a reason for hiding this comment

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

LGTM

Base automatically changed from mkondratek/chore/migrate-api-part-3x to main November 14, 2024 09:45
@mkondratek mkondratek force-pushed the mkondratek/chore/migrate-api-part-4 branch from e3be1ea to f2bdee3 Compare November 14, 2024 09:46
@mkondratek mkondratek enabled auto-merge (squash) November 14, 2024 09:46
@mkondratek mkondratek merged commit 2988bd5 into main Nov 14, 2024
@mkondratek mkondratek deleted the mkondratek/chore/migrate-api-part-4 branch November 14, 2024 09:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants