Skip to content

Teach SectionList.size() and len(sectionlist) to skip deleted sections#3524

Merged
JCGoran merged 2 commits into
masterfrom
fix-3522
Jul 15, 2025
Merged

Teach SectionList.size() and len(sectionlist) to skip deleted sections#3524
JCGoran merged 2 commits into
masterfrom
fix-3522

Conversation

@mgeplf

@mgeplf mgeplf commented Jul 15, 2025

Copy link
Copy Markdown
Collaborator
  • added test

@github-actions

Copy link
Copy Markdown
Contributor

✔️ d82d330 -> artifacts URL

@azure-pipelines

Copy link
Copy Markdown

✔️ d82d330 -> Azure artifacts URL

@codecov

codecov Bot commented Jul 15, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 68.44%. Comparing base (95bd396) to head (e9dcfaf).
⚠️ Report is 56 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3524      +/-   ##
==========================================
- Coverage   68.44%   68.44%   -0.01%     
==========================================
  Files         683      683              
  Lines      116611   116617       +6     
==========================================
+ Hits        79818    79821       +3     
- Misses      36793    36796       +3     

☔ 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.

…tions

* This is a longstanding bug with python's `len(sectionlilst)`
* added test
@github-actions

Copy link
Copy Markdown
Contributor

✔️ 21760ce -> artifacts URL

@azure-pipelines

Copy link
Copy Markdown

✔️ 21760ce -> Azure artifacts URL

Comment thread test/hoctests/tests/test_seclist.py
Comment thread test/hoctests/tests/test_seclist.py
@sonarqubecloud

Copy link
Copy Markdown

@azure-pipelines

Copy link
Copy Markdown

✔️ e9dcfaf -> Azure artifacts URL

@github-actions

Copy link
Copy Markdown
Contributor

✔️ e9dcfaf -> artifacts URL

@JCGoran JCGoran merged commit 3b1cd44 into master Jul 15, 2025
43 checks passed
@JCGoran JCGoran deleted the fix-3522 branch July 15, 2025 17:32
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.

4 participants