Details
-
Bug
-
Status: Closed (View Workflow)
-
Blocker
-
Resolution: Fixed
-
None
Description
This is...stupid. For some reason way back in the day, I set things up such that if there's only one unnamed parameter in a step invocation, that step takes a sole required parameter, and that step takes a body closure, we force-override the parameter type for validation purposes to be String. I seriously have no idea why we did that, and as far as I can tell, it does nothing useful, just blocks things like retry(5).
Attachments
Issue Links
- links to
Activity
| Field | Original Value | New Value |
|---|---|---|
| Description | This is...stupid. For some reason way back in the day, I set things up such that if there's only one unnamed parameter in a step invocation, that step takes a sole required parameter, *and* that step takes a body closure, we force-override the parameter type for validation purposes to be {{String}}. I seriously have no idea why we did that, and as far as I can tell, it does nothing useful, just blocks things like {{retry(5) { ... }}}. | This is...stupid. For some reason way back in the day, I set things up such that if there's only one unnamed parameter in a step invocation, that step takes a sole required parameter, *and* that step takes a body closure, we force-override the parameter type for validation purposes to be {{String}}. I seriously have no idea why we did that, and as far as I can tell, it does nothing useful, just blocks things like {{retry(5)}}. |
| Priority | Critical [ 2 ] | Blocker [ 1 ] |
| Remote Link | This issue links to "PR #104 (Web Link)" [ 15342 ] |
| Status | Open [ 1 ] | In Progress [ 3 ] |
| Status | In Progress [ 3 ] | In Review [ 10005 ] |
| Resolution | Fixed [ 1 ] | |
| Status | In Review [ 10005 ] | Resolved [ 5 ] |
| Status | Resolved [ 5 ] | Closed [ 6 ] |