Fixed minor issues
This commit is contained in:
@@ -41,8 +41,6 @@ def load(path: Path) -> Project:
|
|||||||
raise rethrowme
|
raise rethrowme
|
||||||
|
|
||||||
def _collect_profiles(project: Project) -> None:
|
def _collect_profiles(project: Project) -> None:
|
||||||
assert False, "Not Implemented Yet"
|
|
||||||
|
|
||||||
# TODO: Determine what profile is supposed to be active and load the targets
|
# TODO: Determine what profile is supposed to be active and load the targets
|
||||||
# so that they will be compiled.
|
# so that they will be compiled.
|
||||||
for profile in project.profiles:
|
for profile in project.profiles:
|
||||||
|
|||||||
@@ -3,6 +3,7 @@ from logging import Logger
|
|||||||
|
|
||||||
from ..models import Project
|
from ..models import Project
|
||||||
from ..models import Result
|
from ..models import Result
|
||||||
|
from ..models import Target
|
||||||
from .._config import Config
|
from .._config import Config
|
||||||
from ._commands import Command
|
from ._commands import Command
|
||||||
from ._except import NoTargetsFoundError
|
from ._except import NoTargetsFoundError
|
||||||
@@ -37,9 +38,9 @@ def _prepare_dependencies(
|
|||||||
commands: list[Command] = []
|
commands: list[Command] = []
|
||||||
|
|
||||||
for required in target.deps:
|
for required in target.deps:
|
||||||
dependency = find_target_by_name(required)
|
dependency = find_target_by_name(required.value)
|
||||||
if dependency._prepared:
|
if dependency._prepared:
|
||||||
continue
|
continue
|
||||||
project = find_project_by_target(required)
|
project = find_project_by_target(required.value)
|
||||||
commands.extend(prepare(appcfg, project, target, logger))
|
commands.extend(prepare(appcfg, project, target, logger))
|
||||||
return commands
|
return commands
|
||||||
|
|||||||
@@ -3,6 +3,7 @@ from logging import Logger
|
|||||||
|
|
||||||
from ..models import Project
|
from ..models import Project
|
||||||
from ..models import Result
|
from ..models import Result
|
||||||
|
from ..models import Target
|
||||||
from .._config import Config
|
from .._config import Config
|
||||||
from ._commands import Command
|
from ._commands import Command
|
||||||
from ._commands import ExecuteTest
|
from ._commands import ExecuteTest
|
||||||
@@ -45,9 +46,9 @@ def _prepare_dependencies(
|
|||||||
commands: list[Command] = []
|
commands: list[Command] = []
|
||||||
|
|
||||||
for required in target.deps:
|
for required in target.deps:
|
||||||
dependency = find_target_by_name(required)
|
dependency = find_target_by_name(required.value)
|
||||||
if dependency._prepared:
|
if dependency._prepared:
|
||||||
continue
|
continue
|
||||||
project = find_project_by_target(required)
|
project = find_project_by_target(required.value)
|
||||||
commands.extend(prepare(appcfg, project, target, logger))
|
commands.extend(prepare(appcfg, project, target, logger))
|
||||||
return commands
|
return commands
|
||||||
|
|||||||
Reference in New Issue
Block a user