Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

nil in pipeline-structure breaks step-id calculation leading to mismatches in the UI #172

Closed
flosell opened this issue Oct 30, 2017 · 1 comment
Labels

Comments

@flosell
Copy link
Owner

flosell commented Oct 30, 2017

Steps to reproduce.

  • Add a nil-value to the pipeline structure, e.g.
    (with-frontend-git
            nil
            create-some-details
            client-package
            client-publish)
  • Observe that the UI now does not show output for create-some-details but it's output is visible in the next step:

screen shot 2017-10-30 at 23 17 02

Possible Reasons:

The nil value in the pipeline structure gets filtered out when creating the representation the UI uses but is not filtered when the pipeline gets executed. In the example. create-some-details would be the first child of with-frontend-git in the UI representation but the second child when it comes to calculating the step-id on execution (meaning state will also be written with these step-ids)

Where does this happen in practice?

If you want to create several pipelines from the same structure you might use conditionals that evaluate to nil if not needed in this particular pipeline:

~(when use-trigger?
     `(alias "trigger"
        wait-for-manual-trigger))
@flosell flosell added the bug label Oct 30, 2017
flosell added a commit that referenced this issue Oct 31, 2017
@flosell
Copy link
Owner Author

flosell commented Oct 31, 2017

Released in 0.13.4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant