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

Filter generic base types when seaching for the resource proxy type #510

Merged
merged 2 commits into from
Dec 18, 2024

Conversation

1nf0rmagician
Copy link
Member

@1nf0rmagician 1nf0rmagician commented Dec 10, 2024

When inheriting from a generic base type without adding additional interfaces, a resource can still have a generated proxy. However, in that case the proxy builder would assume the generic base type to be the best possible fit, which cannot be proxified.

Also, adds pre-set package file versions from major version.

Note: As described in #343 we currently don't plan to support generic resource proxies, this only prevents falsely assumed generic types

In order to properly include the projects in different solutions, e.g. to test changes in an application, we need to have the FileVersion set for the projects.
@1nf0rmagician 1nf0rmagician added the bug Something isn't working label Dec 10, 2024
@1nf0rmagician 1nf0rmagician self-assigned this Dec 10, 2024
When inheriting from a generic base type without adding additional interfaces, a resource can still have a generated proxy. However, in that case the proxy builder would assume the generic base type to be the best possible fit, which cannot be proxified.
@1nf0rmagician 1nf0rmagician merged commit 7877c44 into release/6 Dec 18, 2024
1 check passed
@1nf0rmagician 1nf0rmagician deleted the fix/resource-endpoint branch December 18, 2024 05:33
@1nf0rmagician 1nf0rmagician restored the fix/resource-endpoint branch December 18, 2024 05:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants