Config guide edits

Author: treysp

docs/guides/configuration.md

@@ -159,9 +159,11 @@ In some organizations, users never run plans directly against `prod` - they do a
 
 If your organization works like this, it may be convenient to change the `plan` command's default environment from `prod` to each user's development environment. That way people can issue `sqlmesh plan` without typing the environment name every time.
 
-The SQLMesh configuration `user()` function returns the name of the user currently logged in and running SQLMesh. Call it inside Jinja curly braces to use default target environments based on the user name. The curly braces allow you to combine the user name with a prefix or suffix.
+The SQLMesh configuration `user()` function returns the name of the user currently logged in and running SQLMesh. It retrieves the username from system environment variables like `USER` on MacOS/Linux or `USERNAME` on Windows.
 
-The example configuration below will add the username to the end of the string `dev_`. If the user running SQLMesh is `tony`, the default target environment when they run SQLMesh will be `dev_tony`. In other words, `sqlmesh plan` will be equivalent to `sqlmesh plan dev_tony`.
+Call `user()` inside Jinja curly braces with the syntax `{{ user() }}`, which allows you to combine the user name with a prefix or suffix.
+
+The example configuration below constructs the environment name by appending the username to the end of the string `dev_`. If the user running SQLMesh is `tony`, the default target environment when they run SQLMesh will be `dev_tony`. In other words, `sqlmesh plan` will be equivalent to `sqlmesh plan dev_tony`.
 
 === "YAML"
 

sqlmesh/cli/main.py

@@ -1263,20 +1263,6 @@ def state_import(obj: Context, input_file: Path, replace: bool, no_confirm: bool
     obj.import_state(input_file=input_file, clear=replace, confirm=confirm)
 
 
-def _check_engine_installed(console: Console, engine_type: t.Optional[str] = None) -> None:
-    if not engine_type:
-        return
-    connection_config = CONNECTION_CONFIG_TO_TYPE[engine_type]
-
-    try:
-        connection_config._connection_factory.fget(None)
-    except ModuleNotFoundError:
-        install_command = f'pip install "sqlmesh[{engine_type}]"'
-        raise SQLMeshError(
-            f"Unable to load required Python dependencies for the {DISPLAY_NAME_TO_TYPE[engine_type]} engine.\n\nPlease run `{install_command}` to install them before running `sqlmesh init` again."
-        )
-
-
 @cli.group(no_args_is_help=True, hidden=True)
 def dbt() -> None:
     """Commands for doing dbt-specific things"""

tests/core/test_loader.py

@@ -207,7 +207,7 @@ def my_model(context, **kwargs):
 
 
 def test_load_migrated_dbt_adapter_dispatch_macros(tmp_path: Path):
-    init_example_project(tmp_path, dialect="duckdb")
+    init_example_project(tmp_path, engine_type="duckdb")
 
     migrated_package_path = tmp_path / "macros" / c.MIGRATED_DBT_PACKAGES / "dbt_utils"
     migrated_package_path.mkdir(parents=True)
@@ -273,7 +273,7 @@ def test_load_migrated_dbt_adapter_dispatch_macros(tmp_path: Path):
 
 def test_load_migrated_dbt_adapter_dispatch_macros_in_different_packages(tmp_path: Path):
     # some things like dbt.current_timestamp() dispatch to macros in a different package
-    init_example_project(tmp_path, dialect="duckdb")
+    init_example_project(tmp_path, engine_type="duckdb")
 
     migrated_package_path_dbt = tmp_path / "macros" / c.MIGRATED_DBT_PACKAGES / "dbt"
     migrated_package_path_dbt_duckdb = tmp_path / "macros" / c.MIGRATED_DBT_PACKAGES / "dbt_duckdb"