-
Notifications
You must be signed in to change notification settings - Fork 4
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
Make Override a first-class citizen, not just a dict #103
Conversation
3cda9f2
to
f026a78
Compare
Codecov Report
@@ Coverage Diff @@
## master #103 +/- ##
==========================================
+ Coverage 92.15% 92.32% +0.17%
==========================================
Files 60 61 +1
Lines 1771 1798 +27
Branches 298 299 +1
==========================================
+ Hits 1632 1660 +28
Misses 72 72
+ Partials 67 66 -1
... and 1 file with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
0a11fa5
to
53d31f0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
give pipeline node step overrides proper types in JSON Schema and Python
lgtm 💯
|
||
@classmethod | ||
def parse(cls, data: SerializedDict) -> Override: | ||
kwargs = parse_common_step_properties(data) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
reuse from the step.parse
, good good yes yes
Co-authored-by: Aarni Koskela <akx@iki.fi>
53d31f0
to
391dcd8
Compare
Extracted from #97.