Skip to content

Diff editor sync marker by column#5810

Open
anijanyan wants to merge 10 commits intoajaxorg:masterfrom
anijanyan:sync-marker-fix
Open

Diff editor sync marker by column#5810
anijanyan wants to merge 10 commits intoajaxorg:masterfrom
anijanyan:sync-marker-fix

Conversation

@anijanyan
Copy link
Copy Markdown
Contributor

@anijanyan anijanyan commented May 16, 2025

Issue #, if available:

Description of changes:

image

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Pull Request Checklist:

@anijanyan anijanyan changed the title Sync marker fix Diff editor sync marker by column May 16, 2025
@anijanyan anijanyan marked this pull request as ready for review May 20, 2025 13:45
@anijanyan anijanyan marked this pull request as draft May 21, 2025 08:41
@codecov
Copy link
Copy Markdown

codecov bot commented May 28, 2025

Codecov Report

Attention: Patch coverage is 96.61017% with 2 lines in your changes missing coverage. Please review.

Project coverage is 87.78%. Comparing base (0207e6d) to head (a03b6e0).
Report is 8 commits behind head on master.

Files with missing lines Patch % Lines
src/ext/diff/base_diff_view.js 88.88% 1 Missing ⚠️
src/ext/diff/split_diff_view.js 96.96% 1 Missing ⚠️
Additional details and impacted files
@@ Coverage Diff @@ ## master #5810 +/- ## ========================================== + Coverage 87.77% 87.78% +0.01%  ========================================== Files 620 620 Lines 46096 46136 +40 Branches 7559 7562 +3 ========================================== + Hits 40461 40501 +40  Misses 5635 5635 
Flag Coverage Δ
unittests 87.78% <96.61%> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
@anijanyan anijanyan marked this pull request as ready for review June 27, 2025 10:25
# Conflicts: #	src/ext/diff/base_diff_view.js #	src/ext/diff/diff_test.js #	src/ext/diff/styles-css.js #	types/ace-ext.d.ts
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant