Skip to content

feat: Remove DiffViewer component, use CodeEditor to render collapse unchanged readOnly diffView#756

Merged
RohitRaj011 merged 5 commits into
developfrom
feat/replace-diff-viewer
May 28, 2025
Merged

feat: Remove DiffViewer component, use CodeEditor to render collapse unchanged readOnly diffView#756
RohitRaj011 merged 5 commits into
developfrom
feat/replace-diff-viewer

Conversation

@RohitRaj011

@RohitRaj011 RohitRaj011 commented May 27, 2025

Copy link
Copy Markdown
Contributor

Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Checklist

  • The title of the PR states what changed and the related issues number (used for the release note).
  • Does this PR require documentation updates?
  • I've updated documentation as required by this PR.
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas

AbhishekA1509
AbhishekA1509 previously approved these changes May 27, 2025
@RohitRaj011 RohitRaj011 changed the base branch from feat/open-doc-link to develop May 28, 2025 07:06
@RohitRaj011 RohitRaj011 dismissed AbhishekA1509’s stale review May 28, 2025 07:06

The base branch was changed.

@RohitRaj011 RohitRaj011 force-pushed the feat/replace-diff-viewer branch from 7e6362c to 33d7682 Compare May 28, 2025 07:06
@RohitRaj011 RohitRaj011 merged commit 195a393 into develop May 28, 2025
6 checks passed
@RohitRaj011 RohitRaj011 deleted the feat/replace-diff-viewer branch May 28, 2025 07:21
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.

3 participants