Skip to content

Remove unused function _check_yaml() in cli.py #311

@yousefmoazzam

Description

@yousefmoazzam

With the reintroduction of the check command in #201:

httomo/httomo/cli.py

Lines 34 to 46 in ae5610a

@main.command()
@click.argument(
"yaml_config", type=click.Path(exists=True, dir_okay=False, path_type=Path)
)
@click.argument(
"in_data_file",
type=click.Path(exists=True, dir_okay=False, path_type=Path),
required=False, default=None,
)
def check(yaml_config: Path, in_data_file: Path = None):
"""Check a YAML pipeline file for errors."""
in_data = str(in_data_file) if isinstance(in_data_file, PurePath) else None
return validate_yaml_config(yaml_config, in_data)

there's no need for the function _check_yaml() now:

httomo/httomo/cli.py

Lines 182 to 184 in ae5610a

def _check_yaml(yaml_config: Path, in_data: Path):
"""Check a YAML pipeline file for errors."""
return validate_yaml_config(yaml_config, in_data)

Metadata

Metadata

Assignees

No one assigned

    Labels

    minorNice to do but not vital

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions