Skip to content

Conversation

@davesh0812
Copy link
Contributor

πŸ“ Description

Make sure _dict_fields of BaseStep are also included on TaskStep's _dict_fields

βœ… Checklist

  • I updated the documentation (if applicable)
  • I have tested the changes in this PR
  • I confirmed whether my changes are covered by system tests
    • If yes, I ran all relevant system tests and ensured they passed before submitting this PR
    • I updated existing system tests and/or added new ones if needed to cover my changes
  • If I introduced a deprecation:

πŸ§ͺ Testing

Done


πŸ”— References

  • Ticket link:
  • Design docs links:
  • External links:

🚨 Breaking Changes?

  • Yes (explain below)
  • No

πŸ”οΈ Additional Notes

@davesh0812 davesh0812 requested a review from a team as a code owner December 28, 2025 08:07
Copy link
Contributor

@royischoss royischoss left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM πŸ‘

@davesh0812 davesh0812 merged commit c84c80d into mlrun:development Dec 28, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants