Merge branch 'master' into rename-abcs

Author: vdusek

docs/upgrading/upgrading_to_v0x.md

@@ -9,6 +9,10 @@ This page summarizes the breaking changes between Crawlee for Python zero-based
 
 This section summarizes the breaking changes between v0.5.x and v0.6.0.
 
+### Configuration
+
+The `Configuration` fields `chrome_executable_path`, `xvfb`, and `verbose_log` have been removed. The `chrome_executable_path` and `xvfb` fields were unused, while `verbose_log` can be replaced by setting `log_level` to `DEBUG`.
+
 ### Abstract base classes
 
 We decided to move away from [Hungarian notation](https://en.wikipedia.org/wiki/Hungarian_notation) and remove all the `Base` prefixes from the abstract classes. It includes the following public classes:

src/crawlee/_log_config.py

@@ -37,8 +37,6 @@
 def get_configured_log_level() -> int:
     config = service_locator.get_configuration()
 
-    verbose_logging_requested = 'verbose_log' in config.model_fields_set and config.verbose_log
-
     if 'log_level' in config.model_fields_set:
         if config.log_level == 'DEBUG':
             return logging.DEBUG
@@ -53,7 +51,7 @@ def get_configured_log_level() -> int:
 
         assert_never(config.log_level)
 
-    if sys.flags.dev_mode or verbose_logging_requested:
+    if sys.flags.dev_mode:
         return logging.DEBUG
 
     return logging.INFO

src/crawlee/configuration.py

@@ -32,9 +32,6 @@ class Configuration(BaseSettings):
     internal_timeout: Annotated[timedelta | None, Field(alias='crawlee_internal_timeout')] = None
     """Timeout for the internal asynchronous operations."""
 
-    verbose_log: Annotated[bool, Field(alias='crawlee_verbose_log')] = False
-    """Whether to enable verbose logging."""
-
     default_browser_path: Annotated[
         str | None,
         Field(
@@ -244,17 +241,6 @@ class Configuration(BaseSettings):
     ] = './storage'
     """The path to the storage directory. This option is utilized by the `MemoryStorageClient`."""
 
-    chrome_executable_path: Annotated[
-        str | None,
-        Field(
-            validation_alias=AliasChoices(
-                'apify_chrome_executable_path',
-                'crawlee_chrome_executable_path',
-            )
-        ),
-    ] = None
-    """This setting is currently unused."""
-
     headless: Annotated[
         bool,
         Field(
@@ -270,17 +256,6 @@ class Configuration(BaseSettings):
     https://playwright.dev/docs/api/class-browsertype#browser-type-launch.
     """
 
-    xvfb: Annotated[
-        bool,
-        Field(
-            validation_alias=AliasChoices(
-                'apify_xvfb',
-                'crawlee_xvfb',
-            )
-        ),
-    ] = False
-    """This setting is currently unused."""
-
     @classmethod
     def get_global_configuration(cls) -> Self:
         """Retrieve the global instance of the configuration.