Skip to content

Extends variable expanding to maximum depth of 10

Turo Soisenniemi requested to merge (removed):variable_expanding_depth into master

What does this MR do?

Currently variable are expanded only to next depth level. After this MR variables are expanded maximum depth 10.

Why was this MR needed?

Are there points in the code the reviewer needs to double check?

  • Maximum depth is defined for preventing stack overflow (variable expanding is directed graph so it is possible to have cyclic dependency).
    • Should there be warning if maximum depth is met?
  • Because old behaviour is not documented/is undefined, this change can break workflows
    • Should there be configurable maximum and setting it to 1 (previous behaviour) for next release and rising default later
    • Or should there be boolean flag to disable this feature if someone is depending on old behaviour
  • Should Gitlab documentation be updated in different MR?

Does this MR meet the acceptance criteria?

  • Documentation created/updated
  • Tests
    • Added for this feature/bug
    • All builds are passing
  • Branch has no merge conflicts with master (if you do - rebase it please)

What are the relevant issue numbers?

Edited by Turo Soisenniemi

Merge request reports