-
Notifications
You must be signed in to change notification settings - Fork 592
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
Drop support for cs:implements metadata #3247
Comments
This proposal is absolutely doable, but I don't see any advantage to it. But, Ice does support this metadata, and users are probably already using it. |
The only example we provide in the docs is for a local Slice, which makes me thing that was the most common use case. Then just adding and interface to the list of inherited interfaces doesn't seems that useful, you still need to provide an implementation. |
I agree with Jose, I doubt |
Indeed, searching on GitHub shows that the only user of this metadata was us (for local Slice). |
I would also drop |
See also #3246
https://doc.zeroc.com/ice/3.7/the-slice-language/slice-metadata-directives#id-.SliceMetadataDirectivesv3.7-cs:implements:type
Given than C# generated types are always partial types, you can manually declare this additional interfaces in a separate partial struct/class that also provides the implementation.
For example:
The text was updated successfully, but these errors were encountered: