Change error logging to warning for missing playwright and protego imports in bs4_crawler.py (#1536)
<!-- .github/pull_request_template.md --> ## Description <!-- Please provide a clear, human-generated description of the changes in this PR. DO NOT use AI-generated descriptions. We want to understand your thought process and reasoning. --> `bs4_crawler.py` missing imports is not a critical issue. It's not part of core cognee, and can be fixed by installing `pip install "cognee[scraping]"`. Printing `logger.error()` also breaks our integration tests, so this PR uses `logger.warning()` instead ## Type of Change <!-- Please check the relevant option --> - [ ] Bug fix (non-breaking change that fixes an issue) - [ ] New feature (non-breaking change that adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to change) - [ ] Documentation update - [ ] Code refactoring - [ ] Performance improvement - [ ] Other (please specify): ## Screenshots/Videos (if applicable) <!-- Add screenshots or videos to help explain your changes --> ## Pre-submission Checklist <!-- Please check all boxes that apply before submitting your PR --> - [ ] **I have tested my changes thoroughly before submitting this PR** - [ ] **This PR contains minimal changes necessary to address the issue/feature** - [ ] My code follows the project's coding standards and style guidelines - [ ] I have added tests that prove my fix is effective or that my feature works - [ ] I have added necessary documentation (if applicable) - [ ] All new and existing tests pass - [ ] I have searched existing PRs to ensure this change hasn't been submitted already - [ ] I have linked any relevant issues in the description - [ ] My commits have clear and descriptive messages ## DCO Affirmation I affirm that all code in every commit of this pull request conforms to the terms of the Topoteretes Developer Certificate of Origin.
This commit is contained in:
commit
c73e8964a1
1 changed files with 2 additions and 2 deletions
|
|
@ -20,7 +20,7 @@ logger = get_logger(__name__)
|
|||
try:
|
||||
from playwright.async_api import async_playwright
|
||||
except ImportError:
|
||||
logger.error(
|
||||
logger.warning(
|
||||
"Failed to import playwright, make sure to install using pip install playwright>=1.9.0"
|
||||
)
|
||||
async_playwright = None
|
||||
|
|
@ -28,7 +28,7 @@ except ImportError:
|
|||
try:
|
||||
from protego import Protego
|
||||
except ImportError:
|
||||
logger.error("Failed to import protego, make sure to install using pip install protego>=0.1")
|
||||
logger.warning("Failed to import protego, make sure to install using pip install protego>=0.1")
|
||||
Protego = None
|
||||
|
||||
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue