Skip to content

Remove code for old diff strategies #2262

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.

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom
Merged
mrubens merged 1 commit into from
Apr 3, 2025
Merged

Remove code for old diff strategies #2262

mrubens merged 1 commit into from
Apr 3, 2025

Conversation

@mrubens
Copy link
Collaborator

@mrubens mrubens commented Apr 3, 2025 •
edited by ellipsis-dev bot
Loading

Important

Remove old diff strategies and update DiffStrategy.ts to use only MultiSearchReplaceDiffStrategy.

  • Behavior:
    • Removes NewUnifiedDiffStrategy and UnifiedDiffStrategy from the codebase.
    • Updates getDiffStrategy in DiffStrategy.ts to only use MultiSearchReplaceDiffStrategy.
  • Tests:
    • Deletes tests for new-unified and unified strategies in __tests__ directories.
  • Types:
    • Removes types related to new-unified strategy in types.ts.

This description was created by Ellipsis for 69bfd89. It will automatically update as commits are pushed.

@mrubens mrubens requested a review from cte as a code owner April 3, 2025 15:02
@changeset-bot
Copy link

changeset-bot bot commented Apr 3, 2025

⚠️ No Changeset found

Latest commit: 69bfd89

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@dosubot dosubot bot added the size:XXL This PR changes 1000+ lines, ignoring generated files. label Apr 3, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Apr 3, 2025
@mrubens mrubens merged commit 3f5f8bb into main Apr 3, 2025
19 checks passed
@mrubens mrubens deleted the diff_code_cleanup branch April 3, 2025 15:09
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Apr 3, 2025
monotykamary pushed a commit to monotykamary/Roo-Code that referenced this pull request Apr 4, 2025
@saoudrizwan
Fix bug where OpenRouter/Cline providers generation endpoint failed (R…
8cc7759 
…ooCodeInc#2262)

* Fix openrouter/cline provider cost endpoint bug

* Increase generation endpoint timeout
monotykamary pushed a commit to monotykamary/Roo-Code that referenced this pull request Apr 4, 2025
@saoudrizwan
Revert "Fix bug where OpenRouter/Cline providers generation endpoint …
2ca4738 
…failed (RooCodeInc#2262)"

This reverts commit 8cc7759.
monotykamary pushed a commit to monotykamary/Roo-Code that referenced this pull request Apr 4, 2025
@saoudrizwan
Revert "Revert "Fix bug where OpenRouter/Cline providers generation e…
0cd2e69 
…ndpoint failed (RooCodeInc#2262)""

This reverts commit 2ca4738.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cte cte cte approved these changes

Assignees

No one assigned

Labels

lgtm This PR has been approved by a maintainer size:XXL This PR changes 1000+ lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@mrubens @cte