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.

Sign up for GitHub

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

Jump to bottom

Investigate whether `GetMethodInfoUsingManagedNameHelper` should pass `includeNonPublic: true` to `GetRuntimeMethod` #4508

Open

Evangelink opened this issue Jan 4, 2025 · 0 comments

Member

Evangelink commented Jan 4, 2025 •

edited by Youssef1313

Loading

includeNonPublic: false preserves the existing behavior. But I'm suspecting that to be a bug when DiscoverInternals is used, which is why I left it as "TODO". Let's discuss/investigate more on Monday.

Originally posted by @Youssef1313 in #4482 (comment)

The text was updated successfully, but these errors were encountered:

Evangelink mentioned this issue

Allow DynamicData source to be on base types #4482

Merged

Youssef1313 changed the title includeNonPublic: false preserves the existing behavior. But I'm suspecting that to be a bug when DiscoverInternals is used, which is why I left it as "TODO". Let's discuss/investigate more on Monday. Investigate whether GetMethodInfoUsingManagedNameHelper should pass includeNonPublic: true to GetRuntimeMethod

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment