Skip to content

feat: add DocsFilter read path#375

Merged
forshev merged 6 commits into
mainfrom
290-docs-filter-read-path
Apr 10, 2026
Merged

feat: add DocsFilter read path#375
forshev merged 6 commits into
mainfrom
290-docs-filter-read-path

Conversation

@forshev
Copy link
Copy Markdown
Collaborator

@forshev forshev commented Mar 6, 2026

Description

Add read path methods to DocsFilter


  • I have read and followed all requirements in CONTRIBUTING.md;
  • I used LLM/AI assistance to make this pull request;

If you have used LLM/AI assistance please provide model name and full prompt:

Model: {{model-name}}
Prompt: {{prompt}}

@eguguchkin eguguchkin added docs-filter feature New feature or request labels Mar 13, 2026
@eguguchkin eguguchkin added this to the v0.70.0 milestone Mar 16, 2026
@eguguchkin eguguchkin requested review from cheb0 and eguguchkin March 16, 2026 09:42
@forshev forshev force-pushed the 290-docs-filter-read-path branch from 5cb9698 to 7f1ecc3 Compare March 20, 2026 14:21
Comment thread docsfilter/merged_iterator.go Outdated
Comment thread docsfilter/loader.go Outdated
@forshev forshev force-pushed the 290-docs-filter-read-path branch from 7f1ecc3 to 19f4b9d Compare April 8, 2026 10:27
Base automatically changed from 290-docs-filter to main April 10, 2026 12:52
@forshev forshev force-pushed the 290-docs-filter-read-path branch from 19f4b9d to c9f85f5 Compare April 10, 2026 13:01
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 53.64807% with 108 lines in your changes missing coverage. Please review.
✅ Project coverage is 70.09%. Comparing base (6ce5f40) to head (c9f85f5).

Files with missing lines Patch % Lines
filtermanager/loader.go 56.47% 19 Missing and 18 partials ⚠️
filtermanager/filter_manager.go 0.00% 35 Missing ⚠️
filtermanager/iterator_asc.go 65.78% 10 Missing and 3 partials ⚠️
filtermanager/iterator_desc.go 63.88% 10 Missing and 3 partials ⚠️
filtermanager/merged_iterator.go 33.33% 8 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #375      +/-   ##
==========================================
- Coverage   70.32%   70.09%   -0.24%     
==========================================
  Files         214      219       +5     
  Lines       16055    16288     +233     
==========================================
+ Hits        11291    11417     +126     
- Misses       3951     4032      +81     
- Partials      813      839      +26     

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

@forshev forshev merged commit c9dd219 into main Apr 10, 2026
8 checks passed
@forshev forshev deleted the 290-docs-filter-read-path branch April 10, 2026 13:08
@forshev forshev mentioned this pull request Apr 13, 2026
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs-filter feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants