Skip to content
This repository has been archived by the owner on Nov 14, 2019. It is now read-only.

Update process compiler #149

Merged
merged 3 commits into from
Sep 4, 2019
Merged

Update process compiler #149

merged 3 commits into from
Sep 4, 2019

Conversation

antho1404
Copy link
Member

Update the compiler to the latest version:

  • Remove trigger field
  • Rename tasks to steps
  • Include different types of node (filter, trigger, task)

Example of format:

key: Event
steps:
  - type: trigger
    key: trigger
    instance: 
      src: ../serviceA
      env:
        - FOO=BAR
    eventKey: eventX
  - type: filter
    conditions:
      message: event
  - type: task
    key: execute-task-x
    instance:
      src: ../serviceA
      env:
      - FOO=BAR
    taskKey: taskX
    inputs:
      message:
        stepKey: trigger
        key: message
  - type: task
    key: toTaskY
    instance:
      src: ../serviceB
    taskKey: taskY
    inputs:
      inputA:
        stepKey: execute-task-x
        key: xxx

@antho1404 antho1404 added the enhancement New feature or request label Sep 4, 2019
@antho1404 antho1404 added this to the next milestone Sep 4, 2019
@antho1404 antho1404 self-assigned this Sep 4, 2019
Copy link
Member

@NicolasMahe NicolasMahe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If stepKey of an input is empty, the compilation should set it to the previous node/step.

- support of instance resolver
- support of default node key for reference
@antho1404
Copy link
Member Author

Did the update. This adds a bit more complex because we don't want to take the previous node but the previous node that has data (all except the filter)

@NicolasMahe NicolasMahe merged commit fcdb7b1 into master Sep 4, 2019
@NicolasMahe NicolasMahe deleted the feature/new-process-compiler branch September 4, 2019 10:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants