Assets are on by default

# Rust tests and lints will be skipped. Delete if not intended. [ci skip-rust]
thejcannon · Mar 11, 2022 · d588e39 · d588e39
1 parent 209e7ef
commit d588e39
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 8 deletions.
diff --git a/pants.toml b/pants.toml
@@ -115,9 +115,6 @@ macos_big_sur_compatibility = true
 resolves = { python-default = "3rdparty/python/lockfiles/user_reqs.txt" }
 enable_resolves = true
 
-[python-infer]
-assets = true
-
 [docformatter]
 args = ["--wrap-summaries=100", "--wrap-descriptions=100"]
 

diff --git a/src/python/pants/backend/python/dependency_inference/rules.py b/src/python/pants/backend/python/dependency_inference/rules.py
@@ -98,7 +98,7 @@ class PythonInferSubsystem(Subsystem):
     )
     assets = BoolOption(
         "--assets",
-        default=False,
+        default=True,
         help=(
             "Infer a target's asset dependencies based on strings that look like Posix "
             "filepaths, such as those given to `open` or `pkgutil.get_data`. To ignore any "

diff --git a/src/python/pants/backend/python/dependency_inference/rules_test.py b/src/python/pants/backend/python/dependency_inference/rules_test.py
@@ -215,10 +215,7 @@ def test_infer_python_assets(caplog) -> None:
     )
 
     def run_dep_inference(address: Address) -> InferredDependencies:
-        args = [
-            "--source-root-patterns=src/python",
-            "--python-infer-assets",
-        ]
+        args = ["--source-root-patterns=src/python"]
         rule_runner.set_options(args, env_inherit={"PATH", "PYENV_ROOT", "HOME"})
         target = rule_runner.get_target(address)
         return rule_runner.request(