Skip to content

feat: Implement intelligent sequence matching for frame offsets in test comparison #1080

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

Open
DYHARDx wants to merge 1 commit into
base: master
Choose a base branch
from
+68 −64

feat: use difflib SequenceMatcher to handle frame offsets in test diffs

e6c8a3a
Select commit
Loading
Failed to load commit list.
Open

feat: Implement intelligent sequence matching for frame offsets in test comparison #1080

feat: use difflib SequenceMatcher to handle frame offsets in test diffs
e6c8a3a
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Mar 27, 2026 in 43s

Quality Gate passed

Issues
3 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Annotations

Check failure on line 158 in mod_test/nicediff/diff.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define a constant instead of duplicating this literal """ <table>""" 3 times. 

See more on https://sonarcloud.io/project/issues?id=CCExtractor_sample-platform&issues=AZ0t0H9Rcq4s5iMQkB6B&open=AZ0t0H9Rcq4s5iMQkB6B&pullRequest=1080

Check failure on line 179 in mod_test/nicediff/diff.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define a constant instead of duplicating this literal """ </table>""" 3 times. 

See more on https://sonarcloud.io/project/issues?id=CCExtractor_sample-platform&issues=AZ0t0H9Rcq4s5iMQkB6A&open=AZ0t0H9Rcq4s5iMQkB6A&pullRequest=1080

Check failure on line 125 in mod_test/nicediff/diff.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 50 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=CCExtractor_sample-platform&issues=AZ0t0H9Rcq4s5iMQkB6C&open=AZ0t0H9Rcq4s5iMQkB6C&pullRequest=1080
View more details on SonarQubeCloud