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
[WiP] Pipeline validation #20
Conversation
tried this recently - looks 🐝 to me! The validation is nice! We may need to massage the errors so it doesn't say "array shouldn't be 0" though but "a stage must have at least one step" (not sure if part of editor or validation code in declarative though? I think latter) |
@michaelneale yes, I'm going to add special handling for that. the message is coming from PMD validation. I also still need to tweak when the validation gets run, it's a bit too much right now |
🐝 based on screenshots - will run it later, but worth a YOLO merge |
This is a catch-all clean up in preparation for a preview release and validation. Includes pipeline restyling that Brody mocked up.
TODO:
Blurb:
Empty child step placeholders: