Skip to content
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

Fixes bug in super->subclass "auto casting" with union types #752

Closed
wants to merge 2 commits into from

Conversation

tclose
Copy link
Contributor

@tclose tclose commented Apr 24, 2024

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

Summary

  • streamlines typing error messages
  • fixes bug in super->subclass "auto casting" when the sub-classes are part of a union. If the super class is a super class of any of the union args then it passes the check.
  • fixes bug with super->sub-class auto-casting
  • adds handling for MultiInputObj

Checklist

  • I have added tests to cover my changes (if necessary)
  • I have updated documentation (if necessary)

…uto casting" when the sub-classes are part of a union. If the super class is a super class of any of the union args then it is ok.
@tclose tclose added bug Something isn't working patch Increment the patch version when merged (label from auto) release Triggers publishing with auto labels Apr 24, 2024
@tclose tclose requested a review from djarecka April 24, 2024 14:43
@tclose
Copy link
Contributor Author

tclose commented May 21, 2024

Combined with #746 as they are both typing bugfixes and would have merge conflicts

@tclose tclose closed this May 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working patch Increment the patch version when merged (label from auto) release Triggers publishing with auto
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant