Skip to content

Refactor error message handling in get_execution_order to use error objects instead of string concatenation #55

@BaptisteLacroix

Description

@BaptisteLacroix

Description:
Currently, error messages in the get_execution_order method of PipelineEngine (src/jpipe_runner/framework/engine.py) are constructed by concatenating strings. This approach makes it harder to manage, format, and extend error information, especially as the project grows.

Proposed improvement:

  • Create a dedicated error object or exception class to represent pipeline validation errors.
  • Refactor get_execution_order to instantiate and use this error object, encapsulating all relevant details (cycle nodes, labels, suggestions, etc.).
  • Update logging and error reporting to print or serialize the error object, rather than concatenating strings.

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Projects

    Status

    Backlog

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions