Do not compare types, for exact checks use `is` / `is not`, for instance checks use `isinstance()` (E721)

Line too long (125 > 120 characters) (E501)

@Tasks.provides(["dtc.src_dir"]) @Tasks.validate(_validate_dtc) @Tasks.register(category=TaskType.MISC) -def clone_dtc( - context: MlonMcuContext, params=None, rebuild=False, verbose=False, threads=multiprocessing.cpu_count() -): +def clone_dtc(context: MlonMcuContext, params=None, rebuild=False, verbose=False, threads=multiprocessing.cpu_count()): """Clone the dtc repo.""" name = utils.makeDirName("dtc") srcDir = context.environment.paths["deps"].path / "src" / name user_vars = context.environment.vars if "dtc.src_dir" in user_vars: # TODO: also check command line flags?

@Tasks.needs(["dtc.src_dir"]) @Tasks.provides(["dtc.install_dir", "dtc.build_dir", "dtc.exe"]) @Tasks.validate(_validate_dtc) @Tasks.register(category=TaskType.MISC) -def build_dtc( - context: MlonMcuContext, params=None, rebuild=False, verbose=False, threads=multiprocessing.cpu_count() -): +def build_dtc(context: MlonMcuContext, params=None, rebuild=False, verbose=False, threads=multiprocessing.cpu_count()): """Build the device tree compile.""" if not params: params = {} user_vars = context.environment.vars if "dtc.exe" in user_vars: # TODO: also check command line flags?

TestTaskFactory.registry["example_task1"] = mock.Mock(return_value=True) # example_task2_mock = mock.Mock(return_value=True) TestTaskFactory.registry["example_task2"] = mock.Mock(return_value=True) config = {"print_output": print_output} installer = Setup(config=config, context=fake_context, tasks_factory=TestTaskFactory) - result = installer.install_dependencies(progress=progress, write_cache=write_cache, write_env=write_env, rebuild=rebuild) + result = installer.install_dependencies( + progress=progress, write_cache=write_cache, write_env=write_env, rebuild=rebuild + ) assert result # assert example_task1_mock.call_count == 3 assert ( TestTaskFactory.registry["example_task1"].call_count == 1 ) # Due to the mock, the actual wrapper is not executed anymore, params are not considered etc

The following actions uses node12 which is deprecated and will be forced to run on node16: actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/

Input 'skip_existing' has been deprecated with message: The inputs have been normalized to use kebab-case. Use `skip-existing` instead.

Trusted Publishers allows publishing packages to PyPI from automated environments like GitHub Actions without needing to use username/password combinations or API tokens to authenticate with PyPI. Read more: https://docs.pypi.org/trusted-publishers

The following actions uses node12 which is deprecated and will be forced to run on node16: actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/