Skip to content
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.

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

web loader fix #1588

Merged
merged 1 commit into from
Jan 20, 2025
Merged

web loader fix #1588

merged 1 commit into from
Jan 20, 2025

Conversation

pabik
Copy link
Collaborator

@pabik pabik commented Jan 17, 2025

Changes web loader to the correct output.

  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
    Bug fix
  • Why was this change needed? (You can also link to an open issue here)
    Fixes web loader.
  • Other information:

Changes web loader to the correct output.
Copy link

vercel bot commented Jan 17, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs-gpt ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 17, 2025 4:16pm
nextra-docsgpt ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 17, 2025 4:16pm

@github-actions github-actions bot added the application Application label Jan 17, 2025
Copy link

codecov bot commented Jan 17, 2025

Codecov Report

Attention: Patch coverage is 25.00000% with 6 lines in your changes missing coverage. Please review.

Project coverage is 35.44%. Comparing base (13fcbe3) to head (fddee69).
Report is 59 commits behind head on main.

Files with missing lines Patch % Lines
application/parser/remote/web_loader.py 25.00% 6 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1588      +/-   ##
==========================================
- Coverage   35.44%   35.44%   -0.01%     
==========================================
  Files          73       73              
  Lines        3656     3662       +6     
==========================================
+ Hits         1296     1298       +2     
- Misses       2360     2364       +4     

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

@dartpain dartpain merged commit 3de51b6 into main Jan 20, 2025
11 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
application Application
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants